chore(artifacts): dogfood REQ audit — flip REQ-134, reconcile REQ-149 duplicate, file REQ-150#396
Merged
Merged
Conversation
…cile REQ-149 duplicate, file REQ-150 Used `rivet list --type requirement` + `--explain` to audit the REQs filed over recent iterations. Findings, each verified against merged code (not flipped blindly): - REQ-134 (per-project field extension) → implemented: both acceptance criteria are met — the schema-merge field-union mechanism works (verified empirically) and it's documented in docs/schemas.md (#395). - REQ-149 was an accidental duplicate of REQ-134 (filed before noticing the pre-existing one). Linked REQ-149 -> REQ-134 and tagged it so the relationship is explicit rather than a stray parallel item. - Left correctly-draft: REQ-128 (orphans filter shipped, but the inbound-count ranking report is still outstanding), REQ-132 (single-hop link naming done in REQ-147; the multi-hop chain naming is not), REQ-135 (validate/modify enforcement built but inert pending the maintainer's canonical status set + artifact reconciliation). - Filed REQ-150 (draft): `rivet add` warns on no near-duplicate, only duplicate id — the gap that let me file REQ-149. A non-blocking "similar to <ID>" advisory would catch it. `rivet validate` PASS. Implements: REQ-134 Refs: REQ-149, REQ-150, REQ-010 Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
📐 Rivet artifact delta
Graphgraph LR
REQ_134["REQ-134"]:::modified
REQ_149["REQ-149"]:::modified
REQ_150["REQ-150"]:::added
classDef added fill:#d4edda,stroke:#28a745,color:#155724
classDef removed fill:#f8d7da,stroke:#dc3545,color:#721c24
classDef modified fill:#fff3cd,stroke:#ffc107,color:#856404
classDef overflow fill:#e2e3e5,stroke:#6c757d,color:#495057,stroke-dasharray: 3 3
Added
Modified
Posted by |
There was a problem hiding this comment.
⚠️ Performance Alert ⚠️
Possible performance regression was detected for benchmark 'Rivet Criterion Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.20.
| Benchmark suite | Current: 5d55c0b | Previous: 737a99c | Ratio |
|---|---|---|---|
validate/10000 |
17993901 ns/iter (± 3830095) |
14273310 ns/iter (± 228383) |
1.26 |
This comment was automatically generated by workflow using github-action-benchmark.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
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.
A hands-on dogfooding pass: used
rivet list --type requirement+rivet validate --explainto audit the REQs filed over recent iterations and reconcile their statuses. Each finding verified against merged code.Marked implemented
implemented. Both acceptance criteria are met: the schema-merge field-union mechanism works (verified empirically last iteration) and it's documented indocs/schemas.md(docs(schemas): recipe for extending a built-in artifact type with project fields (REQ-149, #350) #395).Duplicate reconciled
REQ-149 → REQ-134(traces-to) + tagged it so it reads as the docs delivery of REQ-134, not a stray parallel requirement.Left correctly draft (verified not fully met)
--orphansfilter shipped (feat(list): --orphans — list artifacts disconnected from the graph (REQ-128) #373), but the inbound-link-count ranking report half is still outstanding.allowed-values+ reconciles artifacts (policy, status lifecycle: enum has no values + promotion gates hardcodeapproved(advancing toimplementedmis-fires; validate vs gaps-json disagree) #355).New friction found + filed
rivet addonly rejects a duplicate id, not a duplicate intent — which is how I filed REQ-149 over REQ-134 without noticing. For the agent-driven workflow (agents filing findings as artifacts) this silently accretes near-duplicates. Proposed a non-blocking "similar to<ID>" advisory. Also filed as a GitHub issue.rivet validatePASS.Implements: REQ-134
🤖 Generated with Claude Code