Skip to content

Commit 3e91552

Browse files
committed
Promote workflow ordering to platform governance and refine Create workflow sequence - PR_26170_008-workflow-order-governance
1 parent 87dfb21 commit 3e91552

15 files changed

Lines changed: 4144 additions & 3276 deletions

docs_build/dev/PROJECT_INSTRUCTIONS.md

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -231,12 +231,13 @@ Allowed intentional-order exceptions:
231231

232232
Every intentional-order exception must be documented in the PR report that introduces or preserves it.
233233

234-
### Toolbox Workflow Ordering Governance
234+
### Workflow Ordering Governance
235235

236-
Toolbox groups are intentional workflow-order exceptions when the surface is guiding creators through a natural making sequence.
236+
When a surface represents a creator workflow, items are ordered by the likely next action, not alphabetically.
237+
Workflow ordering is an approved exception to alphabetical ordering.
238+
This applies to Toolbox, Project Workspace, Create, Publish, Progress, and future guided workflows.
237239

238240
Rules:
239-
- Toolbox group and tile order is by creator progression, not alphabetically.
240241
- Order follows how users naturally work:
241242
- Select -> Create -> Review
242243
- Left -> Right
@@ -246,8 +247,8 @@ Rules:
246247

247248
Create group order:
248249
1. Game Workspace
249-
2. Game Configuration
250-
3. Project Team
250+
2. Project Team
251+
3. Game Configuration
251252
4. Tags
252253

253254
Team planning distinction:

docs_build/dev/admin-notes/tools/index.txt

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
Order,Tool,Group,Status,Complete
22
1,AI Assistant,AI,Wireframe,No
33
2,Game Workspace,Create,Ready,Yes
4-
3,Game Configuration,Create,Ready,Yes
5-
4,Project Team,Create,Planned,No
4+
3,Project Team,Create,Planned,No
5+
4,Game Configuration,Create,Ready,Yes
66
5,Tags,Create,Ready,Yes
77
6,Game Design,Design,Ready,Yes
88
7,Assets,Design,Ready,Yes
@@ -42,17 +42,18 @@ Order,Tool,Group,Status,Complete
4242
41,Game Migration,Platform,Planned,No
4343
42,Platform Settings,Platform,Planned,No
4444

45-
Toolbox Workflow Ordering Governance
46-
- Toolbox groups are intentional workflow-order exceptions.
47-
- Tiles are ordered by creator progression, not alphabetically.
45+
Workflow Ordering Governance
46+
- When a surface represents a creator workflow, items are ordered by likely next action, not alphabetically.
47+
- Workflow ordering is an approved exception to alphabetical ordering.
48+
- Applies to Toolbox, Project Workspace, Create, Publish, Progress, and future guided workflows.
4849
- Order follows how users naturally work: Select -> Create -> Review, Left -> Right, Top -> Bottom.
4950
- The next tile should represent the most likely next creator action.
5051
- Tile ordering should minimize navigation decisions.
5152

5253
Create group order
5354
1. Game Workspace
54-
2. Game Configuration
55-
3. Project Team
55+
2. Project Team
56+
3. Game Configuration
5657
4. Tags
5758

5859
Team distinction

docs_build/dev/reports/PR_26170_006-toolbox-workflow-order-and-project-team.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,13 +6,13 @@
66
## Requirement Checklist
77
- PASS: Built `PR_26170_006-toolbox-workflow-order-and-project-team`.
88
- PASS: Added BUILD doc at `docs_build/pr/BUILD_PR_26170_006-toolbox-workflow-order-and-project-team.md`.
9-
- PASS: Added Toolbox Workflow Ordering Governance to `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
10-
- PASS: Documented Toolbox groups as intentional workflow-order exceptions.
11-
- PASS: Documented tile ordering by creator progression rather than alphabetical sorting.
9+
- PASS: Added Workflow Ordering Governance to `docs_build/dev/PROJECT_INSTRUCTIONS.md`.
10+
- PASS: Documented workflow surfaces as intentional workflow-order exceptions.
11+
- PASS: Documented item ordering by likely next creator action rather than alphabetical sorting.
1212
- PASS: Documented natural ordering rules: `Select -> Create -> Review`, `Left -> Right`, and `Top -> Bottom`.
1313
- PASS: Documented that the next tile should represent the most likely next creator action.
1414
- PASS: Documented that tile ordering should minimize navigation decisions.
15-
- PASS: Updated Create group references to this order: Game Workspace, Game Configuration, Project Team, Tags.
15+
- PASS: Updated Create group references to this order: Game Workspace, Project Team, Game Configuration, Tags.
1616
- PASS: Moved project-specific Users planning into Create/Project Team documentation.
1717
- PASS: Documented the distinction between Studio Team as account-level roster and Project Team as project-level assignment.
1818
- PASS: Updated Owner Notes source content in `docs_build/dev/admin-notes/tools/index.txt`.
@@ -48,7 +48,7 @@
4848
- Full samples smoke: skipped per request and because documentation changes do not affect sample runtime behavior.
4949

5050
## Manual Test Notes
51-
- Open `owner/grouping-colors.html` and confirm Create lists Game Workspace, Game Configuration, Project Team, and Tags.
51+
- Open `owner/grouping-colors.html` and confirm Create lists Game Workspace, Project Team, Game Configuration, and Tags.
5252
- Open Owner Notes for `docs_build/dev/admin-notes/tools/index.txt` and confirm the Create group order and Studio Team / Project Team distinction are visible.
5353
- Confirm no Toolbox page ordering or runtime behavior changed in this PR.
5454

docs_build/dev/reports/PR_26170_007-toolbox-workflow-render-order-project-team.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ PASS - Current branch verified as `main`.
88

99
- PASS - Toolbox grouped rendering no longer alphabetizes cards inside groups. `toolbox/tools-page-accordions.js` uses `compareByWorkflowOrder` for grouped card ordering.
1010
- PASS - Toolbox groups render by declared Game Journey order from source order. The targeted Playwright grouped-view check asserted `GAME_JOURNEY_GROUP_ORDER`.
11-
- PASS - Create group renders in required workflow order: `Game Workspace`, `Game Configuration`, `Project Team`, `Tags`. Covered by source validation and Playwright.
11+
- PASS - Create group renders in required workflow order: `Game Workspace`, `Project Team`, `Game Configuration`, `Tags`. Covered by source validation and Playwright.
1212
- PASS - Existing `users` tool was moved into Create as `Project Team` without changing its route (`toolbox/users/index.html`) or planned status.
1313
- PASS - `Users` is no longer exposed as a Toolbox tile under Share, Account, Platform, or any non-Create group. API snapshot validation confirms `id: users` displays as `Project Team` with `group/category/toolboxGroup: Create`.
1414
- PASS - User-facing Toolbox copy uses `Project Team` for project-specific team assignment. `Studio Team` remains documented as the account-level roster distinction.
@@ -48,6 +48,6 @@ PASS - Current branch verified as `main`.
4848
## Manual Validation Notes
4949

5050
- Confirmed `/toolbox/index.html?view=group` renders all Game Journey groups in declared order.
51-
- Confirmed Create group tile order is `Game Workspace`, `Game Configuration`, `Project Team`, `Tags`.
51+
- Confirmed Create group tile order is `Game Workspace`, `Project Team`, `Game Configuration`, `Tags`.
5252
- Confirmed `Project Team` uses the existing `toolbox/users/index.html` route and remains planned.
5353
- Confirmed `Users` does not remain as a visible Toolbox tile label in the API-backed active registry.
Lines changed: 52 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,52 @@
1+
# PR_26170_008-workflow-order-governance
2+
3+
## Branch Validation
4+
5+
PASS - Current branch verified as `main`.
6+
7+
## Requirement Checklist
8+
9+
- PASS - Renamed the previous Toolbox-specific governance title to `Workflow Ordering Governance` in current governance and owner notes.
10+
- PASS - Updated governance language so creator workflow surfaces order by likely next action, not alphabetically.
11+
- PASS - Documented workflow ordering as an approved exception to alphabetical ordering.
12+
- PASS - Documented applicability to Toolbox, Project Workspace, Create, Publish, Progress, and future guided workflows.
13+
- PASS - Updated Create group order to `Game Workspace`, `Project Team`, `Game Configuration`, `Tags`.
14+
- PASS - Updated Owner grouping notes and Owner Notes source content to the new governance name and Create order.
15+
- PASS - Updated related historical PR documentation references so the old governance name/order does not remain as an active doc reference.
16+
- PASS - Did not move Game Journey. Toolbox mapping still keeps `game-journey` in `Progression`.
17+
- PASS - Did not modify unrelated groups.
18+
19+
## Validation
20+
21+
- PASS - `git branch --show-current` returned `main`.
22+
- PASS - `node --check toolbox/tools-page-accordions.js`
23+
- PASS - `node --check tests/playwright/tools/ToolboxRoutePages.spec.mjs`
24+
- PASS - `git diff --check` completed with no whitespace errors; only Windows line-ending warnings were reported.
25+
- PASS - Targeted Node documentation/static validation confirmed:
26+
- `PROJECT_INSTRUCTIONS.md` uses `Workflow Ordering Governance`.
27+
- likely-next-action and alphabetical-exception rules are present.
28+
- applicability to Toolbox, Project Workspace, Create, Publish, Progress, and future guided workflows is present.
29+
- `PROJECT_INSTRUCTIONS.md`, Owner Notes, Owner grouping notes, Toolbox renderer, and Playwright expectation all use `Game Workspace`, `Project Team`, `Game Configuration`, `Tags`.
30+
- old governance title and old Create order do not remain in searched docs/source targets.
31+
- PASS - Targeted Toolbox Playwright validation:
32+
- `npx playwright test tests/playwright/tools/ToolboxRoutePages.spec.mjs --project=playwright --grep "toolbox grouped view renders Game Journey order" --workers=1 --reporter=list`
33+
- Result: 1 passed.
34+
35+
## Playwright Coverage Note
36+
37+
- Targeted Playwright refreshed advisory V8 coverage reports:
38+
- `docs_build/dev/reports/playwright_v8_coverage_report.txt`
39+
- `docs_build/dev/reports/coverage_changed_js_guardrail.txt`
40+
- The browser run exercised `toolbox/tools-page-accordions.js`.
41+
42+
## Skipped Lanes
43+
44+
- SKIP - Full samples validation was skipped per request.
45+
- SKIP - Broad workspace/runtime suites were skipped because this PR changes governance docs and a targeted Toolbox Create-order renderer mapping only.
46+
- SKIP - Database, persistence, auth, engine, and unrelated tool lanes were skipped because no behavior in those areas changed.
47+
48+
## Manual Notes
49+
50+
- Owner grouping colors now lists Create as `Game Workspace`, `Project Team`, `Game Configuration`, `Tags`.
51+
- Owner Notes source now lists Project Team before Game Configuration in Create.
52+
- Toolbox grouped view keeps Game Journey in `Progression`; only Create tile order changed.
Lines changed: 34 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,20 +1,36 @@
1-
# PR_26170_007 changed files
1+
# git status --short
2+
M docs_build/dev/PROJECT_INSTRUCTIONS.md
3+
M docs_build/dev/admin-notes/tools/index.txt
4+
M docs_build/dev/reports/PR_26170_006-toolbox-workflow-order-and-project-team.md
5+
M docs_build/dev/reports/PR_26170_007-toolbox-workflow-render-order-project-team.md
6+
M docs_build/dev/reports/codex_changed_files.txt
7+
M docs_build/dev/reports/codex_review.diff
8+
M docs_build/dev/reports/coverage_changed_js_guardrail.txt
9+
M docs_build/dev/reports/playwright_v8_coverage_report.txt
10+
M docs_build/pr/BUILD_PR_26170_006-toolbox-workflow-order-and-project-team.md
11+
M docs_build/pr/BUILD_PR_26170_007-toolbox-workflow-render-order-project-team.md
12+
M owner/grouping-colors.html
13+
M tests/playwright/tools/ToolboxRoutePages.spec.mjs
14+
M toolbox/tools-page-accordions.js
15+
?? docs_build/dev/reports/PR_26170_008-workflow-order-governance.md
16+
?? docs_build/pr/BUILD_PR_26170_008-workflow-order-governance.md
217

3-
docs_build/dev/reports/PR_26170_007-toolbox-workflow-render-order-project-team.md
4-
docs_build/dev/reports/codex_changed_files.txt
5-
docs_build/dev/reports/codex_review.diff
6-
docs_build/dev/reports/coverage_changed_js_guardrail.txt
7-
docs_build/dev/reports/playwright_v8_coverage_report.txt
8-
docs_build/pr/BUILD_PR_26170_007-toolbox-workflow-render-order-project-team.md
9-
src/dev-runtime/server/local-api-router.mjs
10-
src/shared/toolbox/tool-metadata-inventory.js
11-
tests/playwright/tools/RootToolsFutureState.spec.mjs
12-
tests/playwright/tools/ToolboxAdminMetadataSsot.spec.mjs
13-
tests/playwright/tools/ToolboxRoutePages.spec.mjs
14-
toolbox/tools-page-accordions.js
15-
toolbox/users/index.html
18+
# git ls-files --others --exclude-standard
19+
docs_build/dev/reports/PR_26170_008-workflow-order-governance.md
20+
docs_build/pr/BUILD_PR_26170_008-workflow-order-governance.md
1621

17-
# Unrelated untracked files intentionally excluded from this PR artifact
18-
19-
docs_build/dev/admin-notes/GFS-Connected-Play-Second-Screen-Turn-Based-Design-26170-001.pdf
20-
docs_build/dev/admin-notes/GFS_Game_Identity_Strategy.pdf.pdf
22+
# git diff --stat
23+
docs_build/dev/PROJECT_INSTRUCTIONS.md | 11 +-
24+
docs_build/dev/admin-notes/tools/index.txt | 15 +-
25+
..._006-toolbox-workflow-order-and-project-team.md | 10 +-
26+
...7-toolbox-workflow-render-order-project-team.md | 4 +-
27+
docs_build/dev/reports/codex_changed_files.txt | 48 +-
28+
docs_build/dev/reports/codex_review.diff | 3572 ++------------------
29+
.../dev/reports/coverage_changed_js_guardrail.txt | 2 +-
30+
.../dev/reports/playwright_v8_coverage_report.txt | 10 +-
31+
..._006-toolbox-workflow-order-and-project-team.md | 12 +-
32+
...7-toolbox-workflow-render-order-project-team.md | 4 +-
33+
owner/grouping-colors.html | 8 +-
34+
tests/playwright/tools/ToolboxRoutePages.spec.mjs | 2 +-
35+
toolbox/tools-page-accordions.js | 4 +-
36+
13 files changed, 382 insertions(+), 3320 deletions(-)

0 commit comments

Comments
 (0)