worktree finish resolves os-sdlc-implementation-report one directory too high, so the usage sweep is silently skipped #797
Labels
No labels
P0
P1
P2
P3
bug
category/code-change
category/cross-repo
category/eval-run
category/investigation
category/no-test-target
category/owner-ruling
category/prose-artifact
category/recurring-chore
category/structure
category/umbrella
crc-refactor
create
delete
enhancement
filed-by/agent
filed-by/user
frozen
lint-rule
needs-info
needs-triage
next
plugin/cc-architect
plugin/os
plugin/os-adr
plugin/os-aidd-lint
plugin/os-backlog
plugin/os-context
plugin/os-doc-hygiene
plugin/os-sdlc
plugin/os-vault
project/cc-os
ready-for-agent
ready-for-human
recurring
review
rule-proposal
update
waiting
wayfinder:grilling
wayfinder:map
wayfinder:map
wayfinder:research
wayfinder:task
wayfinder:task
wontfix
worklist/deviations
worklist/lint-rule
worklist/new-implement-build
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
jared/cc-os#797
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Problem
plugins/os/bin/worktree finish <name> --implementation <id>prints/home/jared/dev/cc-os/os-sdlc/bin/os-sdlc-implementation-report not found, usage not sweptand continues.The script lives at
/home/jared/dev/cc-os/plugins/os-sdlc/bin/os-sdlc-implementation-report.plugins/os/lib/worktree/usage_sweep.rb:8joinsplugin_rootwith.., andplugin_rootalready resolves toplugins/, so the path drops one level too many.The usage sweep is skipped on every finish, and the message is a warning rather than a failure.
Reproduce
From the main checkout, run
plugins/os/bin/worktree finish <any os-sdlc ticket worktree> --implementation <id>and read the summary lines.Origin
VERDICT: CREATE
REASON: plugin_root already resolves to plugins/, so joining '..' drops one level too many; the usage sweep is silently skipped on every finish.
Resolution
Done: duplicate of #733, same defect (relative path to the report script), fixed by ADR-0240
Evidence: merge
6bd034dfFollow-ups: none