This repository was archived by the owner on May 29, 2026. It is now read-only.
chore(openspec): mark applied + archive clean changes + cleanup duplicates#402
Merged
Merged
Conversation
…cates Bookkeeping cleanup of openspec/changes/ on procest: - Marked 36 specs as applied via builds/build.json (deferred-recovery marker for 2026-05-11 batch). - Archived 18 changes that validate strict-clean AND carry the applied marker to openspec/changes/archive/2026-05-11-<name>/. - Removed 3 cross-app accidents (pipelinq-* belong in pipelinq, not procest). - Left 5 umbrella decomposition proposals in place (besluitvorming-workflow, bezwaar-beroep-workflow, gis-integration, vth-workflow-configuration, workflow-engine-enhancement) — these have substantial proposal content (>= 791 words) but no specs/tasks. Flagged for human review rather than silent deletion. Meta PR only — restructuring openspec/ tree plus build.json markers; no app code or runtime changes.
Contributor
Quality Report — ConductionNL/procest @
|
| Check | PHP | Vue | Security | License | Tests |
|---|---|---|---|---|---|
| lint | ✅ | ||||
| phpcs | ❌ | ||||
| phpmd | ✅ | ||||
| psalm | ✅ | ||||
| phpstan | ✅ | ||||
| phpmetrics | ✅ | ||||
| eslint | ❌ | ||||
| stylelint | ❌ | ||||
| composer | ✅ | ✅ 100/100 | |||
| npm | ✅ | ✅ 419/419 | |||
| PHPUnit | ⏭️ | ||||
| Newman | ⏭️ | ||||
| Playwright | ❌ |
Spec coverage: 3% (21 tests / 673 specs)
Quality workflow — 2026-05-11 16:59 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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
Summary
Bookkeeping-only cleanup of
openspec/changes/for procest. Meta PR — no app code, no runtime changes; only openspec/ tree restructuring + build.json markers.Archived (18) — strict-clean validation + applied marker
Moved to
openspec/changes/archive/2026-05-11-<name>/:automatic-actions, beroep-escalation, bezwaar-advisory-committee, bezwaar-decision, bezwaar-hearing, bezwaar-lifecycle, case-location, case-map-overview, enforcement-lhs, inspection-checklists, map-component, parafering-audit-trail, pdok-integration, role-based-step-routing, visual-workflow-editor, voorstel-management, vth-workflow-templates, wms-wfs-layers
Marked applied (36)
Wrote
builds/build.jsonwith{status: applied, applied_at: 2026-05-11, marker: deferred-recovery}to all specs from the deferred-recovery batch that were missing the marker. 12 already had a marker and were skipped.Deleted (3) — cross-app accidents
These belong in the pipelinq app, not procest:
Left open (58) — flagged for human review
Umbrella decomposition proposals (5)
Proposal-only docs (>= 791 words each) without specs/tasks. The earlier audit listed these as empty-shell duplicates, but the content is substantial and matches an umbrella-to-granular decomposition pattern. Per the watchdog instructions, flagged rather than silently deleted:
Validation failures (56)
All 56 failed
openspec validate --strictwith the same reason:Ensure change has deltas in specs/: use headers ## ADDED/MODIFIED/REMOVED/RENAMED Requirements. These need spec-author cleanup before they can be archived. Includes specs that were marked applied (their implementation may be on disk) but whose openspec spec deltas don't match the strict format.Other (1)
unit-test-coverage-75— validates clean but was not in the deferred-recovery batch (no marker), so left open.Test plan