Repository navigation
chore(sigrid): add the fleet's analysis scope file - #1351
Merged
Merged
Conversation
Exclude .claude/** (Claude Code skills and agent config, partly copied from upstream) from the Sigrid analysis scope. Assisted-by: ClaudeCode:claude-opus-5-5
Contributor
Quality Report — ConductionNL/stackiq @
|
| 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
…xclude-vendored-tooling
…xclude-vendored-tooling
This was referenced Nov 2, 2026
Contributor
Quality Report — ConductionNL/stackiq @
|
| 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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds
sigrid.yaml, the same scope file dossiq, pipelinq, shillinq and thematiq carry on development:model: "2026");l10n/*.jscatalogues,scripts/andtools/, 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.phpandcompare_archimate.py. They are developer debug tools (the PHP one boots a local Nextcloud from/var/www/html/lib/base.php), and.distignorealready keepscompare_*.phpout 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)
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 becausescripts/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
check_sig_maintainability.py(gate 122) parses the file and dropsscripts/and bothcompare_archimatefiles from the production set (310 -> 300 files).sigrid.yamlwhen a repository is published to it; whether SIG uses it for an imported assessment is not documented.AI disclosure: prepared with Claude Code (claude-opus-5-5) and reviewed by the lane before opening.