Skip to content

Commit 1a8b4c5

Browse files
author
DavidQ
committed
Document preview tool capabilities and control layout design - PR_26126_002-preview-tool-detailed-design
1 parent 31bccfc commit 1a8b4c5

4 files changed

Lines changed: 57 additions & 534 deletions

File tree

docs/dev/codex_commands.md

Lines changed: 5 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,20 @@
1-
# Codex Commands - PR_26124_077-palette-manager-final-exit-pass
1+
# Codex Commands - PR_26126_002-preview-tool-detailed-design
22

33
```bash
4-
npx @openai/codex run --model gpt-5.5 --reasoning high "Run full workflow for PR_26124_077-palette-manager-final-exit-pass. Follow PROJECT_INSTRUCTIONS.md exactly."
4+
codex run "Create PR_26126_002-preview-tool-detailed-design. Add a detailed design document for tools/preview/index.html covering capabilities, UI regions, control placement, input/output contracts, error states, preview modes, and export actions. No implementation code. Do not modify samples. Place design under docs/pr/PR_26126_002-preview-tool-detailed-design.md and produce required review artifacts."
55
```
66

77
## Validation Commands
88

99
```bash
10-
node --check tools/palette-manager-v2/main.js
11-
node --check tools/palette-manager-v2/paletteManagerShell.js
12-
node --check tools/palette-manager-v2/modules/PaletteManagerApp.js
13-
node tests/tools/PaletteManagerV2Baseline.test.mjs
14-
node --input-type=module <targeted Palette Manager V2 exit-pass audit>
15-
node --input-type=module <targeted Palette Manager V2 URL preset validation>
16-
git diff --check
17-
npm run test:workspace-v2
10+
git diff --check -- docs/pr/PR_26126_002-preview-tool-detailed-design.md docs/dev/codex_commands.md docs/dev/commit_comment.txt
1811
npm run codex:review-artifacts
1912
```
2013

2114
## Playwright
2215

23-
Targeted Palette Manager V2 validation confirms baseline controls, validation clear placement, pin scroll preservation, Tag sort untagged-last behavior, and URL preset loading remain stable.
24-
25-
`npm run test:workspace-v2` failed because `package.json` does not define the `test:workspace-v2` script.
16+
No Playwright impact. This PR is a documentation/design-only PR for the Preview tool and does not modify implementation code.
2617

2718
## Full Samples
2819

29-
Full samples smoke test was skipped by instruction.
20+
Full samples smoke test was skipped because this PR is documentation-only and sample files are out of scope.

docs/dev/commit_comment.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Finalize Palette Manager V2 exit-pass cleanup - PR_26124_077-palette-manager-final-exit-pass
1+
Add Preview tool detailed design - PR_26126_002-preview-tool-detailed-design
Lines changed: 5 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,8 @@
11
# git status --short
2-
M docs/dev/reports/codex_changed_files.txt
3-
M docs/dev/reports/codex_review.diff
4-
D tools/common.zip
5-
D tools/palette-manager-v2.zip
6-
?? docs/pr/PR_26126_002-preview-tool-detailed-design.md
2+
M docs/dev/codex_commands.md
3+
M docs/dev/commit_comment.txt
74

85
# git diff --stat
9-
docs/dev/reports/codex_changed_files.txt | 15 ++---
10-
docs/dev/reports/codex_review.diff | 97 +++----------------------------
11-
tools/common.zip | Bin 1732 -> 0 bytes
12-
tools/palette-manager-v2.zip | Bin 25739 -> 0 bytes
13-
4 files changed, 14 insertions(+), 98 deletions(-)
14-
15-
# PR-scoped diff --stat
16-
.../PR_26126_002-preview-tool-detailed-design.md | 496 +++++++++++++++++++++
17-
1 file changed, 496 insertions(+)
18-
19-
# Note
20-
tools/common.zip and tools/palette-manager-v2.zip were already deleted before this PR task and were not modified by this design-doc change.
6+
docs/dev/codex_commands.md | 19 +++++--------------
7+
docs/dev/commit_comment.txt | 2 +-
8+
2 files changed, 6 insertions(+), 15 deletions(-)

0 commit comments

Comments
 (0)