Skip to content

Commit 2587d07

Browse files
author
DavidQ
committed
PLAN_PR_ENGINE_MATURITY
Defines the next engine-maturity planning layer for the debug platform. Scope includes: - stable debug API boundary - plugin / extension model - external documentation map - versioned contract strategy - performance benchmark rules Docs-only PR. No runtime changes.
1 parent 335fd0c commit 2587d07

13 files changed

+229
-351
lines changed

docs/dev/CODEX_COMMANDS.md

Lines changed: 8 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -2,31 +2,11 @@ MODEL: GPT-5.3-codex
22
REASONING: high
33

44
COMMAND:
5-
Apply BUILD_PR_DEBUG_SURFACES_INSPECTORS exactly as defined.
6-
7-
- Use docs/pr/BUILD_PR_DEBUG_SURFACES_INSPECTORS.md as implementation contract
8-
- Do not expand scope
9-
- Preserve engine/runtime separation
10-
- Do not modify unrelated files
11-
12-
VALIDATION:
13-
- Inspector surfaces are read-only
14-
- Entity inspector works
15-
- Component inspector works
16-
- State diff viewer works
17-
- Timeline debugger works
18-
- Event stream viewer works
19-
- No runtime regressions
20-
- Frame buffer remains bounded
21-
22-
ROADMAP UPDATE:
23-
Update docs/dev/BIG_PICTURE_ROADMAP.md using bracket-only edits:
24-
- Entity inspector -> [x]
25-
- Component inspector -> [x]
26-
- State diff viewer -> [x]
27-
- Timeline debugger -> [x]
28-
- Event stream viewer -> [x]
29-
30-
OUTPUT:
31-
Create APPLY_PR_DEBUG_SURFACES_INSPECTORS delta ZIP at:
32-
<project folder>/tmp/APPLY_PR_DEBUG_SURFACES_INSPECTORS_delta.zip
5+
Create BUILD_PR_ENGINE_MATURITY
6+
7+
REQUIREMENTS:
8+
- Build docs-only engine maturity bundle from PLAN_PR_ENGINE_MATURITY
9+
- Create required PR/dev/report files
10+
- Preserve boundaries and roadmap protection rules
11+
- Bracket-only roadmap state edits when needed
12+
- Package to <project folder>/tmp/BUILD_PR_ENGINE_MATURITY_delta.zip

docs/dev/COMMIT_COMMENT.txt

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1 @@
1-
APPLY_PR_DEBUG_SURFACES_INSPECTORS
2-
3-
Applies the approved inspector layer:
4-
- Entity inspector
5-
- Component inspector
6-
- State diff viewer
7-
- Timeline debugger
8-
- Event stream viewer
9-
10-
Preserves read-only behavior, bounded frame history, optional debug surfaces,
11-
and roadmap bracket-only updates for Track I.
1+
docs: build engine maturity docs bundle from PLAN with stable seams, lifecycle boundaries, versioning, docs ownership, and benchmark rules
Lines changed: 15 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -1,47 +1,21 @@
11
# Engine Maturity API Inventory
22

3-
## Purpose
4-
Define the proven public debug surfaces that may be promoted, and the internal surfaces that remain non-public.
5-
6-
## Promotion Gate (required)
7-
A surface is promotable only if all are true:
8-
1. already used by multiple debug workflows
9-
2. behavior is deterministic and documented
10-
3. does not expose private state shape
11-
4. can be versioned with backward-compatibility notes
12-
13-
## Public Debug API Seams
14-
### Commands
15-
- command pack registration seam
16-
- command discovery/listing seam
17-
- standardized command output contract
18-
19-
### Panels
20-
- panel registration seam
21-
- panel descriptor metadata seam (`id`, `title`, `priority`, `enabled`)
22-
- panel summary render seam
23-
24-
### Providers
25-
- provider registration seam
26-
- read-only snapshot seam
27-
- bounded refresh/poll semantics
28-
29-
### Runtime Control
30-
- debug surface visibility seam (console/overlay)
31-
- deterministic debug render-order seam
32-
33-
## Plugin Lifecycle Seams
34-
- `register(context)`
35-
- `enable(context)`
36-
- `disable(context)`
37-
- `dispose(context)`
38-
39-
## Internal (Do Not Promote)
3+
## Public Seams (Promotable)
4+
- command registration/discovery/execution output contract
5+
- panel registration and descriptor contract
6+
- provider registration and read-only snapshot contract
7+
- debug visibility and deterministic render-order control seams
8+
- plugin lifecycle seam (`register/enable/disable/dispose`)
9+
10+
## Internal (Not Promoted)
4011
- internal overlay composition internals
4112
- private runtime state containers
4213
- persistence internals
43-
- sample-specific wiring and hacks
44-
- experimental inspector internals
14+
- sample-specific hacks/bindings
15+
16+
## Transitional
17+
- compatibility shims for legacy paths only when required
18+
- deprecation notes with replacement path
4519

46-
## Boundary Rule
47-
Promote seams, not implementation details.
20+
## Promotion Gate
21+
Promote only proven, documented, deterministic surfaces that do not leak private state shape.
Lines changed: 7 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1,28 +1,17 @@
11
# Engine Maturity Documentation Map
22

3-
## Purpose
4-
Define ownership and canonical locations for engine-maturity docs.
5-
6-
## Canonical Ownership
7-
### PR History And Phase Artifacts
3+
## Canonical Files
84
- `docs/pr/PLAN_PR_ENGINE_MATURITY.md`
95
- `docs/pr/BUILD_PR_ENGINE_MATURITY.md`
106
- `docs/pr/APPLY_PR_ENGINE_MATURITY.md`
11-
12-
### Active Control Docs
13-
- `docs/dev/codex_commands.md`
14-
- `docs/dev/commit_comment.txt`
15-
- `docs/dev/next_command.txt`
16-
- `docs/dev/BIG_PICTURE_ROADMAP.md`
17-
- `docs/dev/ROADMAP_GUARDRAILS.md`
18-
19-
### Maturity Reference Docs
207
- `docs/dev/ENGINE_MATURITY_API_INVENTORY.md`
218
- `docs/dev/ENGINE_MATURITY_VERSIONING_STRATEGY.md`
229
- `docs/dev/ENGINE_MATURITY_PERFORMANCE_RULES.md`
2310
- `docs/dev/ENGINE_MATURITY_DOCUMENTATION_MAP.md`
2411

25-
## Ownership Rule
26-
- normative migration/stabilization decisions live in `docs/pr/*ENGINE_MATURITY*`
27-
- operational controls live in `docs/dev/*`
28-
- avoid duplicate authoritative statements across files
12+
## Ownership
13+
- `docs/pr`: phase history and implementation intent
14+
- `docs/dev`: active controls + maturity reference docs
15+
16+
## Rule
17+
Do not duplicate normative guidance across multiple files; cross-link to canonical source.
Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,26 +1,17 @@
11
# Engine Maturity Performance Rules
22

3-
## Purpose
4-
Define benchmark entry points and rules for debug-surface maturity.
5-
63
## Benchmark Entry Points
74
1. command execution latency
85
2. overlay render/update frame cost
9-
3. provider snapshot/poll overhead
6+
3. provider polling/snapshot overhead
107
4. panel show/hide/toggle cost
118
5. preset apply/reset latency
129

1310
## Measurement Rules
14-
- compare debug-disabled vs debug-enabled modes
11+
- compare debug-disabled and debug-enabled modes
1512
- measure cold-open and steady-state separately
16-
- capture environment/sample/scenario metadata
17-
- use consistent sample scenes for trend comparison
13+
- record environment/sample/scenario metadata
1814

1915
## Regression Rules
2016
- repeated regressions block maturity promotion
21-
- regressions require mitigation notes before apply closeout
22-
- avoid always-on heavy polling when debug surfaces are idle
23-
24-
## Acceptance
25-
- no user-visible debug latency regressions in normal workflows
26-
- no persistent high-cost background work when debug is disabled
17+
- no always-on heavy polling when debug is idle
Lines changed: 10 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,20 @@
11
# Engine Maturity Versioning Strategy
22

3-
## Purpose
4-
Define contract metadata and lifecycle rules for promoted debug surfaces.
5-
6-
## Contract Metadata (required)
3+
## Contract Metadata
74
Each promoted contract should declare:
85
- `contractId`
96
- `contractVersion`
10-
- `compatibility`
11-
- `backwardCompatible` (boolean)
12-
- `notes` (string)
7+
- `compatibility` (`backwardCompatible`, `notes`)
138
- `status` (`active|deprecated`)
149
- `deprecatedSince` (optional)
1510
- `replacementContractId` (optional)
1611

17-
## Semantic Version Rules
18-
- MAJOR: breaking change
19-
- MINOR: additive compatible change
20-
- PATCH: non-breaking fixes/clarifications
21-
22-
## Compatibility Rules
23-
- MINOR/PATCH must preserve existing consumers
24-
- breaking changes require MAJOR + migration notes
25-
- deprecated contracts require replacement path
12+
## SemVer Rules
13+
- MAJOR: breaking
14+
- MINOR: additive/backward-compatible
15+
- PATCH: non-breaking fix/clarification
2616

27-
## Deprecation Window
28-
1. mark deprecated with replacement
29-
2. keep compatibility during transition window
30-
3. remove only on a MAJOR boundary
17+
## Deprecation
18+
- mark deprecated with replacement guidance
19+
- keep transition window
20+
- remove only on MAJOR boundary

docs/dev/NEXT_COMMAND.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
Create PLAN_PR_ENGINE_MATURITY
1+
APPLY_PR_ENGINE_MATURITY
Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,13 @@
1-
Applies BUILD_PR_DEBUG_SURFACES_INSPECTORS exactly as approved.
1+
Created BUILD_PR_ENGINE_MATURITY docs-only bundle from PLAN.
22

3-
Confirms:
4-
- inspector layer integrated
5-
- read-only inspection preserved
6-
- bounded frame history preserved
7-
- event stream remains passive
8-
- roadmap updated by bracket-only state changes in Track I
3+
Updated:
4+
- BUILD/APPLY PR docs for engine maturity
5+
- maturity API/versioning/performance/docs-map references
6+
- active codex/commit/next-command docs
7+
- reports and file tree
98

10-
No out-of-scope changes permitted.
9+
Constraints enforced:
10+
- docs-only
11+
- no runtime code changes
12+
- preserve public/internal/transitional boundaries
13+
- preserve roadmap guardrails (bracket-only if edited)

docs/dev/reports/file_tree.txt

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,13 @@
1-
docs/pr/APPLY_PR_DEBUG_SURFACES_INSPECTORS.md
2-
docs/dev/codex_commands.md
3-
docs/dev/commit_comment.txt
4-
docs/dev/next_command.txt
5-
docs/dev/reports/change_summary.txt
6-
docs/dev/reports/validation_checklist.txt
7-
docs/dev/reports/file_tree.txt
1+
BUILD_PR_ENGINE_MATURITY_delta.zip
2+
- docs/pr/BUILD_PR_ENGINE_MATURITY.md
3+
- docs/pr/APPLY_PR_ENGINE_MATURITY.md
4+
- docs/dev/ENGINE_MATURITY_API_INVENTORY.md
5+
- docs/dev/ENGINE_MATURITY_VERSIONING_STRATEGY.md
6+
- docs/dev/ENGINE_MATURITY_PERFORMANCE_RULES.md
7+
- docs/dev/ENGINE_MATURITY_DOCUMENTATION_MAP.md
8+
- docs/dev/codex_commands.md
9+
- docs/dev/commit_comment.txt
10+
- docs/dev/next_command.txt
11+
- docs/dev/reports/change_summary.txt
12+
- docs/dev/reports/file_tree.txt
13+
- docs/dev/reports/validation_checklist.txt
Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
1-
[ ] APPLY scope matches approved BUILD scope
2-
[ ] No runtime mutation APIs added
3-
[ ] Entity inspector works
4-
[ ] Component inspector works
5-
[ ] State diff viewer works
6-
[ ] Timeline debugger works
7-
[ ] Event stream viewer works
8-
[ ] Frame buffer bounded
9-
[ ] Existing samples still run
10-
[ ] No unrelated files changed
11-
[ ] BIG_PICTURE_ROADMAP updated with bracket-only edits
1+
Validation Checklist
2+
3+
[x] PLAN_PR -> BUILD_PR -> APPLY_PR flow preserved
4+
[x] Bundle is docs-only
5+
[x] No runtime code files changed in this step
6+
[x] Stable debug API seams documented
7+
[x] Plugin lifecycle boundaries documented
8+
[x] Versioned contract metadata documented
9+
[x] External documentation ownership documented
10+
[x] Performance benchmark entry points/rules documented
11+
[x] Public/internal/transitional boundaries preserved
12+
[x] BIG_PICTURE_ROADMAP.md structure preserved
13+
[x] Bracket-only roadmap edits used if needed

0 commit comments

Comments
 (0)