docs: add Ash Underpass gameplay packet#255
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 82b6dbecf5
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "Codex (@codex) review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "Codex (@codex) address that feedback".
| -> partial_extract | ||
| -> full_clear |
There was a problem hiding this comment.
Split clear outcomes into separate state transitions
The state machine makes partial_extract a mandatory predecessor of full_clear, which conflicts with the later rules that treat them as alternative outcomes after proof_collected. If engineers implement this chain literally, a full-clear run would require passing through partial extraction first, creating ambiguous reward and ledger behavior for mutually exclusive outcomes. This should be modeled as branching transitions from proof_collected (and/or post-boss) rather than a single linear path.
Useful? React with 👍 / 👎.
| TIME result: | ||
|
|
||
| - The Gate should expose 6-10 minutes expected TIME pressure. | ||
| - Exact drain can be simulated first and server-owned later. |
There was a problem hiding this comment.
Keep TIME drain server-authoritative in alpha spec
This line allows TIME drain to be simulated before server ownership, but the same packet later states Unity may not alter TIME and defines server-owned mission outcomes. If the implementation follows client-side TIME first, disconnects or desyncs can produce mismatched death/outcome decisions and violate the authority boundary the doc is trying to enforce. The alpha contract should require server-authoritative TIME accounting from the first playable slice.
Useful? React with 👍 / 👎.
|
|
||
| - Tollkeeper Shell. | ||
| - 2 small adds or hazard pulses. | ||
| - Signal proof route behind or near the arena. |
There was a problem hiding this comment.
Align proof location with partial-extract route
Section 5.4 defines proof collection at Signal Split and uses it to unlock partial extraction, but this requirement places the signal proof route behind or near the boss arena. Implementing this literally can force players into the Toll Plaza path before proof, which undermines the intended interact and extract branch and conflicts with the partial/full outcome design. The proof location should stay consistent with the Signal Split objective contract.
Useful? React with 👍 / 👎.
| - Tollkeeper Shell. | ||
| - 2 small adds or hazard pulses. | ||
| - Signal proof route behind or near the arena. | ||
| - Clear extraction marker after defeat. |
There was a problem hiding this comment.
Make extraction marker available before boss defeat
The room contract says the extraction marker appears after Tollkeeper defeat, but earlier flow/rules define a valid partial-extract path immediately after collecting proof. If this requirement is implemented as written, players cannot perform the advertised safe extraction branch and are forced into the boss route, breaking the core risk-choice behavior and reward split the mission is designed to test.
Useful? React with 👍 / 👎.
Summary
docs/design/54-ash-underpass-gate-gameplay-design.mdas the detailed gameplay packet for the first alpha Gate mission.Verification
npx.cmd --yes markdownlint-cli2 ROADMAP.md docs/SUMMARY.md docs/design/26-alpha-game-design-document.md docs/design/27-alpha-level-design.md docs/design/29-alpha-questline-and-encounter-design.md docs/design/33-alpha-production-backlog.md docs/design/34-alpha-acceptance-matrix.md docs/design/38-alpha-design-decision-register.md docs/design/54-ash-underpass-gate-gameplay-design.md: pass, 0 errors.git diff --check: pass.rg -n "—|–" ...changed docs...: pass, no generated em dash/en dash.tools/windows/check-backlog-sync.ps1 -NoGitHub: WARN only for existing broad ROADMAP TODOs without nearby issue refs; tracked files check passed.Local reviewer fallback
.claude/CLAUDE.mdchanges.