Skip to content

Commit 07127c4

Browse files
author
DavidQ
committed
Rollback Palette Manager V2 shared removal attempt to last stable state - PR_26124_072-palette-manager-rollback-shared-removal
1 parent 94edc91 commit 07127c4

8 files changed

Lines changed: 286 additions & 646 deletions

File tree

docs/dev/codex_commands.md

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,26 @@
1-
# Codex Commands - PR_26124_071-palette-manager-remove-tools-shared-shell
1+
# Codex Commands - PR_26124_072-palette-manager-rollback-shared-removal
22

33
```bash
4-
npx @openai/codex run --model gpt-5.5 --reasoning high "Run full workflow for PR_26124_071-palette-manager-remove-tools-shared-shell. Follow PROJECT_INSTRUCTIONS.md exactly."
4+
npx @openai/codex run --model gpt-5.5 --reasoning high "Run full workflow for PR_26124_072-palette-manager-rollback-shared-removal. Follow PROJECT_INSTRUCTIONS.md exactly."
55
```
66

77
## Validation Commands
88

99
```bash
10+
git status --short
11+
git diff -- tools/palette-manager-v2
1012
node --check tools/palette-manager-v2/paletteManagerShell.js
1113
node --check tools/palette-manager-v2/main.js
12-
node --input-type=module <tools/shared shell dependency validation>
13-
node --input-type=module <css structural validation>
14-
node --input-type=module <targeted Palette Manager local shell Playwright validation>
15-
node tests/tools/ToolLayoutDockingControlNormalization.test.mjs
16-
node tests/tools/ToolEntryLaunchContract.test.mjs
17-
git diff --check -- tools/palette-manager-v2/index.html tools/palette-manager-v2/paletteManagerV2.css tools/palette-manager-v2/paletteManagerShell.js
14+
git diff --check
1815
npm run test:workspace-v2
16+
npm run codex:review-artifacts
1917
```
2018

2119
## Playwright
2220

23-
Targeted Palette Manager V2 local shell Playwright validation passed. It confirmed the page no longer loads `platformShell` assets, local header/status render, hide/show summary behavior works, Palette Manager controls render, and menuSample actions remain centered.
21+
No Palette Manager behavior change is expected. The default requested gate is `npm run test:workspace-v2`.
2422

25-
`npm run test:workspace-v2` was also requested as the default gate. It failed because `package.json` does not define the `test:workspace-v2` script.
23+
`npm run test:workspace-v2` failed because `package.json` does not define the `test:workspace-v2` script.
2624

2725
## Full Samples
2826

docs/dev/commit_comment.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Remove Palette Manager dependency on tools shared shell - PR_26124_071-palette-manager-remove-tools-shared-shell
1+
Rollback uncommitted Palette Manager shared-removal attempt - PR_26124_072-palette-manager-rollback-shared-removal
Lines changed: 30 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,30 @@
1+
# PR_26124_072 Palette Manager Rollback Shared Removal
2+
3+
## Summary
4+
The uncommitted shared-removal attempt was already absent from tracked Palette Manager V2 files. The rollback pass confirmed `tools/palette-manager-v2` matches `HEAD` and removed leftover ignored/untracked artifacts from the abandoned attempt.
5+
6+
## Runtime Files
7+
- No tracked `tools/palette-manager-v2` runtime files changed in this rollback PR.
8+
- `git diff --exit-code -- tools/palette-manager-v2` passed.
9+
10+
## Removed Local Artifacts
11+
- `tmp/PR_26124_072-palette-manager-shared-removal-review-fix_delta.zip`
12+
- Empty untracked directory `docs/pr/PR_26124_072-palette-manager-shared-removal-review-fix/`
13+
14+
## Files Added Or Updated For Workflow
15+
- `docs/pr/PR_26124_072-palette-manager-rollback-shared-removal/PLAN_PR.md`
16+
- `docs/pr/PR_26124_072-palette-manager-rollback-shared-removal/BUILD_PR.md`
17+
- `docs/pr/PR_26124_072-palette-manager-rollback-shared-removal/APPLY_PR.md`
18+
- `docs/dev/codex_commands.md`
19+
- `docs/dev/commit_comment.txt`
20+
- `docs/dev/reports/codex_review.diff`
21+
- `docs/dev/reports/codex_changed_files.txt`
22+
23+
## Validation
24+
- PASS: `git status --short`
25+
- PASS: `node --check tools/palette-manager-v2/paletteManagerShell.js`
26+
- PASS: `node --check tools/palette-manager-v2/main.js`
27+
- PASS: `git diff --exit-code -- tools/palette-manager-v2`
28+
- PASS: `git diff --check`
29+
- FAIL: `npm run test:workspace-v2` is unavailable because `package.json` does not define a `test:workspace-v2` script.
30+
- SKIPPED: full samples smoke test, by instruction.
Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,10 @@
11
# git status --short
22
M docs/dev/codex_commands.md
33
M docs/dev/commit_comment.txt
4-
A docs/dev/reports/PR_26124_071_report.md
5-
A docs/pr/PR_26124_071-palette-manager-remove-tools-shared-shell/APPLY_PR.md
6-
A docs/pr/PR_26124_071-palette-manager-remove-tools-shared-shell/BUILD_PR.md
7-
A docs/pr/PR_26124_071-palette-manager-remove-tools-shared-shell/PLAN_PR.md
8-
M tools/palette-manager-v2/index.html
9-
A tools/palette-manager-v2/paletteManagerShell.js
10-
M tools/palette-manager-v2/paletteManagerV2.css
4+
A docs/dev/reports/PR_26124_072_rollback_report.md
5+
A docs/pr/PR_26124_072-palette-manager-rollback-shared-removal/APPLY_PR.md
6+
A docs/pr/PR_26124_072-palette-manager-rollback-shared-removal/BUILD_PR.md
7+
A docs/pr/PR_26124_072-palette-manager-rollback-shared-removal/PLAN_PR.md
118

129
# git diff --stat
1310
(no output)

0 commit comments

Comments
 (0)