Skip to content

chore(sigrid): add the fleet's analysis scope file - #1351

Merged
rubenvdlinde merged 4 commits into
developmentfrom
chore/sigrid-exclude-vendored-tooling
Nov 2, 2026
Merged

rubenvdlinde merged 4 commits into
developmentfrom
chore/sigrid-exclude-vendored-tooling

Conversation

@rubenvdlinde

@rubenvdlinde rubenvdlinde commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

What

Adds sigrid.yaml, the same scope file dossiq, pipelinq, shillinq and thematiq carry on development:

  • pins the maintainability model (model: "2026");
  • lists every technology in the repo with its test-code patterns (listing languages switches off automatic detection, so the list names them all);
  • leaves out only generated files and developer tooling: the generated l10n/*.js catalogues, scripts/ and tools/, OpenSpec change proposals, .claude/, and the root tool configuration (ESLint, PHPStan bootstrap, PHP-CS-Fixer).

One stackiq-specific line: the root ArchiMate comparison scripts compare_archimate.php and compare_archimate.py. They are developer debug tools (the PHP one boots a local Nextcloud from /var/www/html/lib/base.php), and .distignore already keeps compare_*.php out of the release package.

This branch first held a minimal file that only excluded .claude/; it now holds the full file, with development merged in.

Effect on the gate 122 proxy (development 1c3dff9; gate 123 on the merged head)

without sigrid.yaml with this file
checks met, 4-star 28 / 48 14 / 34
checks met, 5-star 22 / 48 8 / 34

The drop is the scope change, not the code: the only Python file is compare_archimate.py, so all 14 Python checks (all met) leave scope, as they did for dossiq and pipelinq. No other check changes state. Within the remaining checks the shares move a little because scripts/ held small, simple units: php params >=5 4.7% -> 4.8%, javascript CCN >25 11.1% -> 11.5%. Both were missed before and still are; the open SIG PRs #1370, #1371 and #1372 take javascript CCN >25 to 0% either way.

Checked

AI disclosure: prepared with Claude Code (claude-opus-5-5) and reviewed by the lane before opening.

Exclude .claude/** (Claude Code skills and agent config, partly copied
from upstream) from the Sigrid analysis scope.

Assisted-by: ClaudeCode:claude-opus-5-5
@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/stackiq @ 825a7ef

Check PHP Vue Security License Tests
lint ✅
phpcs ✅
phpmd ✅
psalm ✅
phpstan ✅
phpmetrics ✅
eslint ✅
stylelint ✅
build ✅
check-manifest ✅
check-vue-demi ✅
test-l10n ✅
format ❌
check-schema-l10n ❌
check-l10n-js ✅
composer ❌ ✅ 127/127
npm ❌ ✅ 807/807
app:check-code ⏭️
info.xml ✅
REUSE ❌
lockfile sync ✅
PHPUnit ⏭️ not run for this diff — no file in this diff matches the code globs, and none carries a source extension — the heavy tier has nothing to decide about it.
Newman ⏭️
Playwright ⏭️ deferred: E2E runs locally and on the promotion path only. This pull request targets development, so the suite is asked once per promotion into beta and main rather than once per push per open pull request. Run it locally with npx playwright test, or from the Actions tab on a branch with no open pull request into development.
Hydra gates ❌

Quality workflow — 2026-11-01 20:34 UTC

Download the full PDF report from the workflow artifacts.

Same file as dossiq, pipelinq, shillinq and thematiq: pins the model,
lists every technology with its test patterns, and leaves generated l10n
catalogues and developer tooling out (scripts/, tools/, openspec change
proposals, .claude/, root tool configuration). Stackiq adds its root
ArchiMate comparison scripts (compare_archimate.php and .py), which
.distignore already keeps out of the release package.

Assisted-by: ClaudeCode:claude-opus-5-5
@rubenvdlinde rubenvdlinde changed the title chore(sigrid): leave vendored tooling out of the analysis scope chore(sigrid): add the fleet's analysis scope file Nov 2, 2026
@github-actions

github-actions Bot commented Nov 2, 2026

Copy link
Copy Markdown
Contributor

Quality Report — ConductionNL/stackiq @ c22966d

Check PHP Vue Security License Tests
lint ✅
phpcs ✅
phpmd ✅
psalm ✅
phpstan ✅
phpmetrics ✅
eslint ✅
stylelint ✅
build ✅
check-manifest ✅
check-vue-demi ✅
test-l10n ✅
format ❌
check-schema-l10n ❌
check-l10n-js ✅
composer ❌ ✅ 127/127
npm ❌ ✅ 807/807
app:check-code ⏭️
info.xml ✅
REUSE ❌
lockfile sync ✅
PHPUnit ⏭️ not run for this diff — no file in this diff matches the code globs, and none carries a source extension — the heavy tier has nothing to decide about it.
Newman ⏭️
Playwright ⏭️ deferred: E2E runs locally and on the promotion path only. This pull request targets development, so the suite is asked once per promotion into beta and main rather than once per push per open pull request. Run it locally with npx playwright test, or from the Actions tab on a branch with no open pull request into development.
Hydra gates ❌

Quality workflow — 2026-11-02 20:55 UTC

Download the full PDF report from the workflow artifacts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant