Skip to content

Commit e4b78f5

Browse files
maxrittermaxritter
committed
feat: plan annotations, code review mode, E2E test scenarios in spec workflow
Console — Plan Annotation UI (Spec tab): - Block-based markdown rendering with native Selection API in Specifications tab - Select any text in annotate mode → write free-text note → auto-saves to JSON immediately - Single annotation type (free-text only — no DELETION/INSERTION/REPLACEMENT sub-types) - Removed type toolbar and 'Add general comment' button - View/Review toggle next to Specifications title; smart default (annotate for unapproved PENDING, view otherwise) - AnnotationPanel, AnnotationToolbar, BlockRenderer, PlanAnnotator, useAnnotation, parser components Console — Code Review Mode (Changes tab): - View/Review toggle moved to Changes page header (previously buried in DiffPanel) - Inline diff annotations: click '+' on any hunk line, write note, auto-saves to unified JSON - Staged/unstaged counts in file list sidebar header - CodeReviewPanel, CodeReviewWidget, useCodeReview components extracted Console — Unified Annotation Storage: - Single JSON per spec at docs/plans/.annotations/{plan-filename}.json - Both planAnnotations and codeReviewAnnotations in one file - Auto-save on every change (debounced) — no 'Send Feedback' button anywhere - Removed feedback markdown file generation entirely - AnnotationRoutes.ts and annotationStore.ts refactored to unified model - Active spec resolution for Changes tab scopes code review annotations per spec Console — Sidebar + Layout: - Sidebar reorder: Dashboard → Changes → Specifications → Extensions → ... - Changes and Specifications now adjacent for the plan→review→verify flow Spec Commands — E2E Test Scenarios: - spec-plan Step 1.5.2: write structured TS-NNN scenarios during planning for UI features (step table, priority, preconditions, mapped tasks — agent-browser executable) - Task structure gains 'Mapped Scenarios' field; plan template gains '## E2E Test Scenarios' section - spec-verify Step 3.9 refactored: executes plan's TS-NNN scenarios with TaskCreate tracking, fix attempts (max 2 → known issue escalation), results written to '## E2E Results' in plan file - spec-bugfix-plan: Verification Scenario for UI-facing bugs (single TS-001 acceptance scenario) - spec-bugfix-verify Step 3.5b: executes verification scenario, loops back on 2-attempt failure - Console Spec view: E2E Test Scenarios, E2E Results, Verification Scenario added to DISPLAYED_SECTIONS with icons Spec Commands — Annotation JSON Integration: - spec-plan Step 1.7b and spec-bugfix-plan Step 1.4b read planAnnotations from unified JSON - spec-verify Step 3.12b and spec-bugfix-verify Step 3.7b read codeReviewAnnotations from unified JSON - All annotation feedback steps updated to DELETE via new /api/annotations endpoints Launcher — Worktree Sync Auto-Stash: - sync_worktree() now auto-stashes uncommitted changes on base branch before merging - Stash is preserved (never dropped) if restore encounters conflicts - spec-verify and spec-bugfix-verify: removed pre-sync working tree cleanliness check - Fixes GitHub issue #119 ('Main branch has an unstashed file') Rules + Commands — playwright-cli → agent-browser: - pilot/rules/playwright-cli.md removed; pilot/rules/agent-browser.md added - All spec commands, rules, and settings updated to reference agent-browser Installer: - finalize.py: kill stale Console worker (port 41777) on install so new files take effect immediately - dependencies.py: agent-browser replaces playwright-cli in dependency checks - shell_config.py: encoding improvements Other: - LICENSE: Enterprise Source Code License clause for 100+ seat organizations - settings.json: autoDreamEnabled; removed CLAUDE_CODE_MAX_OUTPUT_TOKENS/EFFORT_LEVEL overrides; agent-browser tip - pilot/ui/: viewer bundle rebuilt with annotation components (PlanAnnotator.js) - README, Docusaurus, site: updated spec workflow descriptions, E2E scenarios, console features - docs/docusaurus: remote-control.md removed, console.md expanded with annotation features
1 parent 641c0e4 commit e4b78f5

88 files changed

Lines changed: 1081 additions & 809 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

‎.gitignore‎

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,8 @@ __marimo__/
224224
# Claude Code
225225
CLAUDE.md
226226
AGENTS.md
227+
spec-annotate
228+
changes-header
227229
apm.yml
228230
.playwright-*/
229231
.apm
@@ -247,6 +249,7 @@ playwright/
247249

248250
# Build artifacts
249251
docs/plans
252+
docs/plans/.annotations/
250253
docs/license
251254
docs/video
252255
docs/slides

‎LICENSE‎

Lines changed: 17 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,18 @@ revocable license to:
7070
and may be used by You for any purpose, including commercial purposes,
7171
in perpetuity, even after Your Subscription ends.
7272

73+
(e) ENTERPRISE SOURCE CODE LICENSE. Subject to an active Enterprise
74+
Subscription, the Licensor grants You the additional right to receive,
75+
copy, modify, and create Derivative Works of the complete Software source
76+
code, including Compiled Components, solely for Your internal business
77+
purposes. This includes the right to fork, adapt, and deploy customized
78+
versions of the Software within Your organization. This right does not
79+
include the right to redistribute, sublicense, sell, or make available
80+
any such Derivative Works to third parties outside Your organization.
81+
The Enterprise Source Code License is available only to organizations
82+
with one hundred (100) or more developer seats under a separate Enterprise
83+
agreement with the Licensor.
84+
7385
--------------------------------------------------------------------------------
7486

7587
3. TRIAL PERIOD
@@ -94,9 +106,11 @@ You shall NOT, and shall not permit any third party to:
94106
non-commercial purposes;
95107

96108
(b) CREATE OR DISTRIBUTE DERIVATIVE WORKS based on the Software or any
97-
portion thereof. You may modify the Source Code for Your own internal use
98-
as permitted under Section 2(c), but You may not distribute, publish, or
99-
make available any Derivative Work to any third party;
109+
portion thereof, except as expressly permitted under Section 2(e)
110+
(Enterprise Source Code License). You may modify the Source Code for Your
111+
own internal use as permitted under Section 2(c), but You may not
112+
distribute, publish, or make available any Derivative Work to any third
113+
party;
100114

101115
(c) REBRAND, rename, or present the Software or any portion thereof as Your
102116
own product, or remove, alter, or obscure any proprietary notices,

‎README.md‎

Lines changed: 30 additions & 15 deletions

‎console/package.json‎

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,8 @@
4848
"autoprefixer": "^10.4.23",
4949
"daisyui": "^5.5.14",
5050
"esbuild": "^0.27.2",
51+
"highlight.js": "^11.11.1",
52+
"marked": "^17.0.5",
5153
"postcss": "^8.5.6",
5254
"react": "^18.3.1",
5355
"react-dom": "^18.3.1",

‎console/src/cli/handlers/user-message.ts‎

-197 Bytes
Binary file not shown.

‎console/src/services/worker-service.ts‎

147 Bytes
Binary file not shown.

‎console/src/services/worker/http/routes/AnnotationRoutes.ts‎

5.19 KB
Binary file not shown.

‎console/src/services/worker/http/routes/LicenseRoutes.ts‎

326 Bytes
Binary file not shown.

‎console/src/services/worker/http/routes/PlanRoutes.ts‎

-766 Bytes
Binary file not shown.

‎console/src/services/worker/http/routes/ViewerRoutes.ts‎

193 Bytes
Binary file not shown.

0 commit comments

Comments
 (0)