This repository was archived by the owner on May 29, 2026. It is now read-only.
retrofit: annotate openregister Bucket 1 (395 methods / 92 REQs / 98 files)#1337
Merged
WilcoLouwerse merged 7 commits intoApr 28, 2026
Merged
Conversation
Generated by opsx-coverage-scan v1 on 2026-04-23. The change proposal captures Bucket 1 high-confidence REQ matches (395 methods across 98 files) for retroactive @SPEC annotation on retrofit/annotate-openregister-2026-04-23. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds @SPEC PHPDoc tags across 65 lib/ files, pointing methods at their matching REQs via openspec/changes/retrofit-annotate-openregister-2026-04-23/tasks.md. Pure annotation change — no logic modifications. Source: Bucket 1 high-confidence matches from opsx-coverage-scan v1 (2026-04-23). NEEDS-REVIEW entries deferred for human triage. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The initial file landed with a placeholder hash for 197da46. Real SHA: 197da46fc6e1f8a... (full in file). No behavior change — just makes the blame-ignore file actually work. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The ghost change is complete (all 92 tasks marked [x] as retroactive annotations). Move to openspec/archive/ to match OpenSpec workflow. No spec deltas — no capability specs modified. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Contributor
Quality Report — ConductionNL/openregister @
|
| Check | PHP | Vue | Security | License | Tests |
|---|---|---|---|---|---|
| lint | ✅ | ||||
| phpcs | ✅ | ||||
| phpmd | ✅ | ||||
| psalm | ✅ | ||||
| phpstan | ✅ | ||||
| phpmetrics | ✅ | ||||
| eslint | ✅ | ||||
| stylelint | ✅ | ||||
| composer | ✅ | ✅ 147/147 | |||
| npm | ✅ | ✅ 599/599 | |||
| PHPUnit | ❌ | ||||
| Newman | ❌ | ||||
| Playwright | ⏭️ |
Quality workflow — 2026-04-24 05:24 UTC
Download the full PDF report from the workflow artifacts.
rjzondervan
approved these changes
Apr 24, 2026
rjzondervan
left a comment
rjzondervan
left a comment
Member
There was a problem hiding this comment.
Approved, but to prevent it from intervering with the active review on dev -> beta, wait with merging until that is done.
This was referenced Apr 24, 2026
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
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.
Retrofit — Annotation Only
Applies
@spec openspec/changes/retrofit-annotate-openregister-2026-04-23/tasks.md#task-Ntags per ADR-003 §Spec traceability.Ghost change:
openspec/archive/retrofit-annotate-openregister-2026-04-23/(empty spec delta, 92 tasks, all[x]as retroactive).What this PR does
@spectags across 98 files (74 PHP + 24 frontend)@spectags where missing (21 frontend files got new top-of-file JSDoc blocks).git-blame-ignore-revsWhat this PR does NOT do
/opsx-reverse-specPRs)Verification
composer phpcspasses (0 errors across 71 files)npm run lintpasses (0 errors, 218 warnings — all from eslint-plugin-jsdoc not knowing the custom@spectag; config-level follow-up)@spectag (grep-verified).git-blame-ignore-revsincludes both annotation commits (197da46 + 5e0d5a8)Capabilities covered
21 capabilities — top 5 by method count: audit-trail-immutable (117), workflow-engine-abstraction (37), faceting-configuration (35), zoeken-filteren (27), archival-destruction-workflow (23).
Enabling blame-ignore locally
Per-developer choice (not auto-applied).
Source
openspec/coverage-report.mdgenerated 2026-04-23Test plan
composer phpcspasses in CInpm run lintpasses in CI🤖 Generated with Claude Code