Skip to content

Commit cb32730

Browse files
ToolboxAidToolboxAid
committed
Use the individual commit comment generated for each PR. Do not squash PR_26152_109 through PR_26152_113 into a single commit.
1 parent b554391 commit cb32730

24 files changed

Lines changed: 1295 additions & 58 deletions

‎docs/dev/reports/project_workspace_contract_rename_validation.md‎

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,42 +1,42 @@
1-
# Project Workspace Contract Rename Validation
1+
# ProjectWorkspace Contract Rename Validation
22

33
PR: PR_26152_079-project-workspace-contract-rename
44
Date: 2026-06-02
55

66
## Scope
77

88
- Continued from PR_26152_078.
9-
- Renamed runtime-only Workspace terminology to Project Workspace.
9+
- Renamed runtime-only Workspace terminology to ProjectWorkspace.
1010
- Updated contracts, contract tests, docs/specs, and required reports only.
1111
- No database implementation, authentication implementation, runtime UI, page, CSS, or HTML changes were made.
1212

1313
## Contract Rename
1414

1515
- The previous runtime-only contract file was renamed to `src/shared/contracts/projectWorkspaceRuntimeContract.js`.
1616
- The matching runtime-only contract test was renamed to `tests/shared/ProjectWorkspaceRuntimeContract.test.mjs`.
17-
- The runtime-only contract terminology was renamed to Project Workspace Runtime Contract.
18-
- Project Workspace is the active runtime working context for a Project.
17+
- The runtime-only contract terminology was renamed to ProjectWorkspace Runtime Contract.
18+
- ProjectWorkspace is the active runtime working context for a Project.
1919

20-
## Project Workspace Rules
20+
## ProjectWorkspace Rules
2121

22-
- Project Workspace is runtime-only.
23-
- Project Workspace does not persist tool payloads.
24-
- Project Workspace does not own saved tool state.
25-
- Project Workspace does not duplicate project storage.
26-
- Project Workspace does not duplicate tool state storage.
27-
- Project Workspace recovery points to Tool State recovery.
22+
- ProjectWorkspace is runtime-only.
23+
- ProjectWorkspace does not persist tool payloads.
24+
- ProjectWorkspace does not own saved tool state.
25+
- ProjectWorkspace does not duplicate project storage.
26+
- ProjectWorkspace does not duplicate tool state storage.
27+
- ProjectWorkspace recovery points to Tool State recovery.
2828

2929
## Data Ownership
3030

3131
- Project = persisted DB container.
32-
- Project Workspace = current runtime working context for the Project.
32+
- ProjectWorkspace = current runtime working context for the Project.
3333
- Tool State = persisted DB record for one tool.
3434
- Manifest = portable export/import format.
3535

3636
## Docs
3737

3838
- Added `docs/dev/specs/PROJECT_WORKSPACE_RUNTIME_CONTRACT.md`.
39-
- Updated `docs/dev/specs/TOOL_LAUNCH_SSOT.md` to use Project Workspace terminology for prior runtime launch state.
39+
- Updated `docs/dev/specs/TOOL_LAUNCH_SSOT.md` to use ProjectWorkspace terminology for prior runtime launch state.
4040

4141
## Validation Commands
4242

@@ -68,8 +68,8 @@ rg '"workspaceState"' -n src/shared/contracts/projectWorkspaceRuntimeContract.js
6868

6969
## Expected Results
7070

71-
- PASS: Project Workspace contract tests validate terminology and behavior remain unchanged.
72-
- PASS: Project Workspace rejects old standalone workspace storage fields such as `workspaceState`.
71+
- PASS: ProjectWorkspace contract tests validate terminology and behavior remain unchanged.
72+
- PASS: ProjectWorkspace rejects old standalone workspace storage fields such as `workspaceState`.
7373
- PASS: Tool State remains the saved editing source.
7474
- PASS: Project, Identity/Permissions, and Tool contract tests remain compatible.
7575
- PASS: no runtime/UI/CSS/HTML files changed.

‎docs/dev/reports/projectworkspace_launch_boundary_validation.md‎

Lines changed: 57 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,57 @@
1+
# ProjectWorkspace Launch Boundary Validation
2+
3+
PR: PR_26152_111-projectworkspace-launch-boundary-validation
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Added ProjectWorkspace launch boundary validation.
9+
- Validated selected tool launch uses explicit manifest and toolState inputs.
10+
- Validated invalid launch inputs reject visibly.
11+
- Did not validate tool runtime behavior.
12+
- Did not require any tool to pass runtime UAT.
13+
14+
## Validation
15+
16+
Command:
17+
18+
```powershell
19+
node tests/shared/ProjectWorkspaceLaunchBoundaryValidation.test.mjs
20+
```
21+
22+
Result: PASS.
23+
24+
## Coverage
25+
26+
- PASS: valid launch input uses explicit ProjectWorkspace, manifest, and toolState inputs.
27+
- PASS: missing manifest input rejects visibly.
28+
- PASS: missing toolState input rejects visibly.
29+
- PASS: hidden fallback data rejects visibly.
30+
- PASS: unmigrated tool runtime is SKIP / not migrated / out of scope, not FAIL.
31+
32+
## Lanes Executed
33+
34+
- ProjectWorkspace launch contract boundary validation.
35+
36+
## Lanes Skipped
37+
38+
- tool runtime validation - future lane.
39+
- samples - SKIP / pending rebuild.
40+
- Playwright - not impacted.
41+
42+
## Samples Decision
43+
44+
SKIP / pending rebuild. No sample JSON was read as source of truth or modified.
45+
46+
## Playwright
47+
48+
Playwright impacted: No.
49+
50+
## Blocker Scope
51+
52+
No blockers.
53+
54+
## Manual Validation
55+
56+
- Confirmed validation rejects invalid boundary input before any tool runtime requirement.
57+
- Confirmed skipped unmigrated tools are not treated as failures.

‎docs/dev/reports/projectworkspace_manifest_handoff_boundary_validation.md‎

Lines changed: 57 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,57 @@
1+
# ProjectWorkspace Manifest Handoff Boundary Validation
2+
3+
PR: PR_26152_112-projectworkspace-manifest-handoff-validation
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Added ProjectWorkspace manifest handoff boundary validation.
9+
- Validated declared manifest handoff rules only.
10+
- Validated invalid manifest payloads reject before downstream use.
11+
- Did not validate tool-specific rendering or behavior.
12+
13+
## Validation
14+
15+
Command:
16+
17+
```powershell
18+
node tests/shared/ProjectWorkspaceManifestHandoffBoundaryValidation.test.mjs
19+
```
20+
21+
Result: PASS.
22+
23+
## Coverage
24+
25+
- PASS: tools consume declared manifest fields only.
26+
- PASS: undeclared downstream manifest fields reject before downstream use.
27+
- PASS: tool runtime payload leakage rejects before render.
28+
- PASS: invalid manifest contract payload rejects before render.
29+
- PASS: tool-specific rendering remains out of scope.
30+
31+
## Lanes Executed
32+
33+
- ProjectWorkspace manifest handoff contract boundary validation.
34+
35+
## Lanes Skipped
36+
37+
- tool rendering validation - future lane.
38+
- tool runtime validation - future lane.
39+
- samples - SKIP / pending rebuild.
40+
- Playwright - not impacted.
41+
42+
## Samples Decision
43+
44+
SKIP / pending rebuild. No sample JSON was modified or used as fallback data.
45+
46+
## Playwright
47+
48+
Playwright impacted: No.
49+
50+
## Blocker Scope
51+
52+
No blockers.
53+
54+
## Manual Validation
55+
56+
- Confirmed invalid handoffs reject before downstream tool use.
57+
- Confirmed validation does not require tool rendering behavior.

‎docs/dev/reports/projectworkspace_state_boundary_validation.md‎

Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
# ProjectWorkspace State Boundary Validation
2+
3+
PR: PR_26152_113-projectworkspace-state-boundary-validation
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Added ProjectWorkspace state boundary validation.
9+
- Validated ProjectWorkspace state remains coordination-only.
10+
- Validated Tool State owns saved payload state.
11+
- Validated palette ownership boundaries.
12+
- Validated hidden persisted ProjectWorkspace assumptions reject.
13+
- Did not validate tool runtime behavior.
14+
15+
## Validation
16+
17+
Command:
18+
19+
```powershell
20+
node tests/shared/ProjectWorkspaceStateBoundaryValidation.test.mjs
21+
```
22+
23+
Result: PASS.
24+
25+
## Coverage
26+
27+
- PASS: ProjectWorkspace remains coordination-only.
28+
- PASS: Tool State remains the saved editing source.
29+
- PASS: active palette context references palette/tool state only.
30+
- PASS: ProjectWorkspace cannot own Tool State records.
31+
- PASS: ProjectWorkspace cannot rely on localStorage or sessionStorage.
32+
- PASS: ProjectWorkspace cannot carry hidden fallback data.
33+
34+
## Lanes Executed
35+
36+
- ProjectWorkspace state contract boundary validation.
37+
38+
## Lanes Skipped
39+
40+
- tool runtime validation - future lane.
41+
- samples - SKIP / pending rebuild.
42+
- Playwright - not impacted.
43+
44+
## Samples Decision
45+
46+
SKIP / pending rebuild. No sample files were touched.
47+
48+
## Playwright
49+
50+
Playwright impacted: No.
51+
52+
## Blocker Scope
53+
54+
No blockers.
55+
56+
## Manual Validation
57+
58+
- Confirmed ProjectWorkspace state has no hidden persisted workspace storage assumptions.
59+
- Confirmed Tool State and palette payload ownership remain outside ProjectWorkspace.

‎docs/dev/reports/projectworkspace_terminology_alignment.md‎

Lines changed: 57 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,57 @@
1+
# ProjectWorkspace Terminology Alignment
2+
3+
PR: PR_26152_109-projectworkspace-terminology-alignment
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Aligned current docs, reports, fixtures, and validation language to use ProjectWorkspace as the preferred term.
9+
- Retained Workspace V2 only for existing file names, command names, package scripts, or historical references.
10+
- No runtime behavior changes.
11+
- No tool runtime tests run.
12+
- No samples touched.
13+
14+
## Changes
15+
16+
- Updated ProjectWorkspace runtime contract validation messages.
17+
- Updated ProjectWorkspace runtime spec wording.
18+
- Updated current contract validation labels that referred to Project Workspace.
19+
- Added ProjectWorkspace validation boundary guidance at `docs/dev/specs/PROJECTWORKSPACE_VALIDATION_BOUNDARIES.md`.
20+
21+
## Validation
22+
23+
Command:
24+
25+
```powershell
26+
git diff --check
27+
```
28+
29+
Result: PASS.
30+
31+
## Lanes Executed
32+
33+
- contract/docs boundary language validation only.
34+
35+
## Lanes Skipped
36+
37+
- runtime - no runtime behavior changed.
38+
- tool runtime - future lane.
39+
- samples - SKIP / pending rebuild.
40+
- recovery/UAT - not part of this boundary lane.
41+
42+
## Samples Decision
43+
44+
SKIP / pending rebuild. No sample JSON or sample runtime files were touched.
45+
46+
## Playwright
47+
48+
Playwright impacted: No.
49+
50+
## Blocker Scope
51+
52+
No blockers.
53+
54+
## Manual Validation
55+
56+
- Confirmed ProjectWorkspace is the preferred current term in new validation language.
57+
- Confirmed Workspace V2 remains only where file names, commands, or historical references require it.

‎docs/dev/reports/projectworkspace_validation_boundaries.md‎

Lines changed: 58 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,58 @@
1+
# ProjectWorkspace Validation Boundaries
2+
3+
PR: PR_26152_110-projectworkspace-validation-boundaries
4+
Date: 2026-06-02
5+
6+
## Scope
7+
8+
- Corrected the active ProjectWorkspace validation boundary.
9+
- Marked samples as SKIP / pending rebuild.
10+
- Marked unmigrated tools as SKIP / not migrated / out of scope.
11+
- Marked tool runtime validation as a future lane.
12+
- Kept ProjectWorkspace lifecycle and contract validation as the only active scope.
13+
14+
## Boundary Rules
15+
16+
- ProjectWorkspace contract-valid inputs are PASS.
17+
- Invalid ProjectWorkspace boundary inputs are REJECT with visible validation detail.
18+
- Unmigrated tools are SKIP and are not classified as FAIL.
19+
- Tool runtime validation is future lane.
20+
- Samples are SKIP / pending rebuild.
21+
22+
## Validation
23+
24+
Command:
25+
26+
```powershell
27+
git diff --check
28+
```
29+
30+
Result: PASS.
31+
32+
## Lanes Executed
33+
34+
- contract/docs boundary validation only.
35+
36+
## Lanes Skipped
37+
38+
- runtime - no runtime behavior changed.
39+
- tool runtime - future lane.
40+
- samples - SKIP / pending rebuild.
41+
- Playwright - not impacted.
42+
43+
## Samples Decision
44+
45+
SKIP / pending rebuild. No sample files were changed or validated.
46+
47+
## Playwright
48+
49+
Playwright impacted: No.
50+
51+
## Blocker Scope
52+
53+
No blockers. Unmigrated tools remain explicitly outside this lane.
54+
55+
## Manual Validation
56+
57+
- Confirmed unmigrated tools are documented as SKIP / not migrated / out of scope.
58+
- Confirmed no report language classifies unmigrated tools as FAIL.

‎docs/dev/specs/GAME_MANIFEST_CONTRACT.md‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ This document does not:
104104
- implement manifest persistence
105105
- implement authentication
106106
- implement authorization checks
107-
- implement Project Workspace storage
107+
- implement ProjectWorkspace storage
108108
- implement runtime manifest loading
109109
- implement publishing
110110
- implement marketplace behavior

0 commit comments

Comments
 (0)