[Bug] /opsx:verify accepts a near-equivalent heading as the REMOVED baseline, so a replacement requirement is reported as still implemented
I maintainer di solito rispondono entro 2 giorni
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Idoneità per principianti
- 72/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Attiva
- Stack tecnologico
- markdown, typescript
- Ambito
- devtools
Direzione di ricerca
La formulazione errata si trova nel passaggio 5 (Spec Coverage) di .claude/commands/opsx/verify.md, nel punto elenco che inizia con "If that file holds no such requirement under this or a near-equivalent name". Lo stesso testo è riportato in .claude/skills/openspec-verify-change/SKILL.md ed è generato da dist/core/templates/workflows/verify-change.js, quindi controlla anche il template sorgente. Il lavoro è completo quando i requisiti REMOVED vengono associati solo tramite intestazione normalizzata, quando si ricorre alla cronologia git nel caso in cui tale intestazione manchi e quando i file generati vengono aggiornati di conseguenza.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
Since the fix for #2067, /opsx:verify reads the baseline of a REMOVED requirement from the main spec before searching the codebase. The new text accepts a "near-equivalent" heading as the baseline. When the REMOVED requirement has been replaced by a similarly named ADDED requirement and /opsx:sync has already updated the main spec, the near-equivalent heading is the replacement, not the removed requirement. The verifier then searches for the replacement's behavior, finds it, and reports the removed requirement as still implemented.
Version
@fission-ai/openspec 1.14.2, Claude Code delivery (skills + commands), spec-driven schema.
Where
.claude/commands/opsx/verify.md, step 5 Spec Coverage, the "For each REMOVED requirement" bullet that begins "If that file holds no such requirement under this or a near-equivalent name", and the mirrored .claude/skills/openspec-verify-change/SKILL.md. In the package the text comes from dist/core/templates/workflows/verify-change.js.
The bullet before it already says to match "by normalized name"; the "near-equivalent" wording in the next bullet contradicts that and decides the fallback to history.
Repro
- Main spec has
### Requirement: Monthly export in xlsx. - A change has a delta with
## REMOVED RequirementsforMonthly export in xlsxand## ADDED RequirementsforMonthly export in xlsx as attendance sheet. /opsx:syncis run, so the main spec now holds onlyMonthly export in xlsx as attendance sheet.- Run
/opsx:verify.
Expected: the exact normalized REMOVED heading is absent from the main spec, so the baseline is recovered from the file's git history (the removed requirement's own body and scenarios), and the search is for that behavior.
Actual: the replacement heading counts as a near-equivalent name, so its body is used as the baseline. The code legitimately implements it, so the removal is reported as not done.
Why it matters
The failure is a false "still implemented" on a removal that was done correctly, driven by a legitimate pattern (replace a requirement by a more specific one). It only appears after a sync, which is exactly the case the history fallback was added for.
Suggested fix
Match the REMOVED heading by its normalized name only. If the main spec has no requirement with that normalized heading, go to the history fallback; do not accept a similarly named heading. If a rename is involved, that is the RENAMED operation's job.
- Lingua principale
- TypeScript
- Stelle
- 71.6k
- Fork
- 4.9k
- Merge medio
- 2g 6h
- PR unite (30g)
- 82
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Fission-AI/OpenSpec
-
Workflow templates: only update-change keeps a chosen artifact path inside changeRoot; continue, propose and ff do notForse già presa @intrepid37 l’ha presa oggi. Apertabug needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Fission-AI/OpenSpec#2099 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Fission-AI/OpenSpec#1966 · 1 commento ·
I maintainer di solito rispondono entro 2 giorni
-
feat(schemas): open each proposal with a short review focusForse già presa @GaoSSR l’ha presa 27 giorni fa. Apertadesign-review
Difficoltà 1/5 1-3 ore Idoneità per principianti 90/100
Fission-AI/OpenSpec#1908 ·
I maintainer di solito rispondono entro 2 giorni
-
A `spec.md` symlink to a file elsewhere in the same project is refused since 1.8.0Forse già presa @clay-good l’ha presa 1 giorno fa. Apertaapproved
Difficoltà 3/5 Mezza giornata Idoneità per principianti 30/100
Fission-AI/OpenSpec#2100 · 1 reazione ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 38/100
Fission-AI/OpenSpec#2088 ·
I maintainer di solito rispondono entro 2 giorni
Tutte le issue di Fission-AI/OpenSpec
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno
-
bug category:ai category:devices effort:s
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
LanternOps/breeze#8943 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
-
area:reliability difficulty:easy help wanted points:100 Stellar Wave
Difficoltà 2/5 1-3 ore Idoneità per principianti 80/100
Divinity-Labs/Lens#341 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno