Skip to content

W43 — World-graph content-definition contract - #119

Merged
The-Running-Dev merged 8 commits into
mainfrom
feature/w43-world-graph-content-contract-plan
Aug 3, 2026
Merged

W43 — World-graph content-definition contract#119
The-Running-Dev merged 8 commits into
mainfrom
feature/w43-world-graph-content-contract-plan

Conversation

@The-Running-Dev

@The-Running-Dev The-Running-Dev commented Aug 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • combine the W43 design plan and its complete contract execution in one PR
  • replace the world-graph content noun list with complete source/runtime campaign schemas
  • define typed maps, terrain, scenery, products, buildings, agents, scenarios, rules, conditions, effects, defaults, and validation
  • reconcile W42 runtime state only where typed content proves a missing or duplicated consumer
  • include a minimum Sun Trap-shaped source and representative invalid-source findings

Scope and dependency

Documentation only. No file under src/engine/ changes.

This PR targets main. W44 is now consolidated in #120 and stacked on this exact branch; after #119 merges, #120 should be retargeted to main.

Validation

  • strict documentation checks passed for the affected files
  • full repository documentation check passed across 177 Markdown files during contract execution
  • npm --prefix site run check passed for the planning changes
  • git diff --check

@The-Running-Dev
The-Running-Dev marked this pull request as ready for review August 2, 2026 23:09
@qodo-code-review

Copy link
Copy Markdown
Contributor

PR Summary by Qodo

Add W43 world-graph content contract execution plan and update ledgers

📝 Documentation ⚙️ Configuration changes 🕐 20-40 Minutes

Grey Divider

AI Description

• Add a doc-only execution plan for the W43 world-graph content-definition contract.
• Reconcile W42 as merged and update programme/TODO ledgers to mark W43 ready.
• Update the public roadmap link to point directly at the focused W43 plan.
Diagram

graph TD
  A["docs/docs/engine/TODO.md"] -->|"marks W43 ready"| C["plans/43-w43-world-graph-content-contract.md"] -->|"plan to execute"| D["docs/docs/engine/12-world-graph-kind.md"]
  B["plans/39-world-graph-kind-programme.md"] -->|"links W43 plan"| C -->|"updates execution brief"| D
  E["site/src/roadmap/roadmapData.ts"] -->|"public link"| C -->|"guides work in"| D
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Inline W43 plan into the programme doc (plans/39) only
  • ➕ Single place to read the whole programme without jumping files
  • ➕ Avoids a new standalone document to maintain
  • ➖ Makes the programme doc unwieldy and harder to use as a ledger
  • ➖ Loses a crisp 'Handoff — Start Here' execution brief for W43
2. Track W43 solely in issues/Projects instead of a repo plan doc
  • ➕ Easier task-level assignment and progress tracking
  • ➕ Better visibility for non-doc readers
  • ➖ Harder to capture schema decisions, gates, and validation matrix in one durable artifact
  • ➖ Breaks the repo’s existing contract-unit planning pattern

Recommendation: Keep the standalone W43 plan document as implemented. It matches the repo’s contract-unit workflow, provides an executable brief (handoff + gates), and allows the programme/TODO ledgers and public roadmap to link to a stable, focused source of truth without mixing long-form design guidance into ledger documents.

Files changed (4) +371 / -18

Documentation (3) +369 / -16
TODO.mdMark W42 merged and W43 planned/ready with plan link +11/-6

Mark W42 merged and W43 planned/ready with plan link

• Updates the engine TODO ledger to record W42 as completed/merged (with PR reference). Replaces the prior W43 'proposed' entry with a 'planned' entry and links directly to the W43 execution plan as the brief.

docs/docs/engine/TODO.md

39-world-graph-kind-programme.mdReconcile W42 checklist completion and point W43 at its plan +12/-10

Reconcile W42 checklist completion and point W43 at its plan

• Checks off the W42 runtime-state contract tasks to reflect the merged work. Adds a direct reference to the new W43 plan and marks the 'cut the plan' programme item as done.

plans/39-world-graph-kind-programme.md

43-w43-world-graph-content-contract.mdAdd W43 doc-only execution plan for the content-definition contract +346/-0

Add W43 doc-only execution plan for the content-definition contract

• Introduces a comprehensive W43 plan defining scope, working rules, key design decisions, required type inventory, execution sequence, validation tiers, and done-when gates. Establishes the intended reconciliation boundary with W42 and explicitly excludes engine code and W44 resolution design.

plans/43-w43-world-graph-content-contract.md

Other (1) +2 / -2
roadmapData.tsUpdate roadmap link to point at the W43 contract plan +2/-2

Update roadmap link to point at the W43 contract plan

• Changes the public roadmap chapter link label and URL to reference the W43 plan document rather than the broader world-graph programme plan.

site/src/roadmap/roadmapData.ts

@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo Fixer

No findings are available for this PR yet. Findings appear here once Qodo has reviewed the PR.

@The-Running-Dev

Copy link
Copy Markdown
Owner Author

/review

Comment thread plans/43-w43-world-graph-content-contract.md Outdated
Comment thread plans/43-w43-world-graph-content-contract.md
@The-Running-Dev The-Running-Dev changed the title Plan W43 world-graph content contract W43 — World-graph content-definition contract Aug 3, 2026
Comment thread docs/docs/engine/12-world-graph-kind.md
Comment thread docs/docs/engine/12-world-graph-kind.md
Comment thread docs/docs/engine/12-world-graph-kind.md
Comment thread docs/docs/engine/TODO.md
Comment thread docs/docs/guide.md Outdated
@The-Running-Dev
The-Running-Dev merged commit e300074 into main Aug 3, 2026
4 checks passed
@The-Running-Dev
The-Running-Dev deleted the feature/w43-world-graph-content-contract-plan branch August 3, 2026 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant