Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
id: bugfix-839
title: review-comment-commands-missin
protocol: bugfix
phase: verified
plan_phases: []
current_plan_phase: null
gates: {}
iteration: 1
build_complete: false
history: []
started_at: '2026-05-26T05:44:32.129Z'
updated_at: '2026-05-26T05:51:41.231Z'
32 changes: 32 additions & 0 deletions codev/state/bugfix-839_thread.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,32 @@
# bugfix-839 thread

## Investigate

Issue #839: two `contributes.commands` entries in `packages/vscode/package.json` are missing the `Codev:` prefix that every other command uses.

Located the two entries at `packages/vscode/package.json:213-220`:

- `codev.submitReviewComment` → `"Submit review comment"`
- `codev.deleteReviewComment` → `"Delete review comment"`

Grep for these strings across `packages/vscode/` finds them only in `package.json` — no other code/docs reference the title text, so renaming is safe (commands are referenced by their internal id, not the title).

Fix: title-case to match the rest of the command set:
- `Codev: Submit Review Comment`
- `Codev: Delete Review Comment`

Net diff: 2 lines.

No regression test possible/needed — this is a static `package.json` manifest entry surfaced by VS Code's marketplace UI. There is no runtime path to assert against; the only "test" is reading the JSON. Will note this in the review.

## PR

PR #864 opened. Build + tests green via porch checks.

## CMAP (iter 1)

- gemini: APPROVE — HIGH confidence
- codex: APPROVE — "Scoped manifest fix matches the issue and leaves all 40 contributed command titles consistently prefixed with `Codev:`."
- claude: APPROVE — "Minimal, correct fix — two command titles prefixed to match convention; all 40 commands verified consistent; no test needed for static manifest entries."

3/3 APPROVE. Notifying architect.
4 changes: 2 additions & 2 deletions packages/vscode/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -211,12 +211,12 @@
},
{
"command": "codev.submitReviewComment",
"title": "Submit review comment",
"title": "Codev: Submit Review Comment",
"enablement": "!commentIsEmpty"
},
{
"command": "codev.deleteReviewComment",
"title": "Delete review comment",
"title": "Codev: Delete Review Comment",
"icon": "$(trash)"
}
],
Expand Down
Loading