waypoint-codex 0.15.0 → 0.16.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
package/README.md CHANGED
@@ -1,53 +1,153 @@
1
1
  # Waypoint
2
2
 
3
- Waypoint is a collaborator-first repository operating system for Codex.
3
+ Waypoint makes Codex better by default for real software work.
4
4
 
5
- It exists to solve two problems at the same time:
5
+ Codex is already powerful. The problem is that most users still have to teach it
6
+ the same things over and over:
6
7
 
7
- - the next agent should be able to pick up the repo with real context
8
- - the current agent should still feel smart, direct, and useful
8
+ - ask better questions before coding
9
+ - plan thoroughly instead of making hidden assumptions
10
+ - follow stronger coding standards
11
+ - review the result seriously before merge
12
+ - verify the work instead of guessing
13
+ - learn from corrections instead of repeating the same mistakes
9
14
 
10
- ## What Waypoint is for
15
+ Waypoint installs those defaults into a repo so you can spend less time
16
+ prompting and more time building.
11
17
 
12
- Waypoint is built for repos where continuity and collaboration both matter.
18
+ ## Install and upgrade
13
19
 
14
- It gives the next session real context and keeps the current session clear and useful.
20
+ Waypoint requires Node 20+.
21
+
22
+ Install globally:
23
+
24
+ ```bash
25
+ npm install -g waypoint-codex
26
+ ```
27
+
28
+ Or try it without a global install:
29
+
30
+ ```bash
31
+ npx waypoint-codex@latest --help
32
+ ```
33
+
34
+ Keep an existing repo up to date:
35
+
36
+ ```bash
37
+ waypoint upgrade
38
+ ```
39
+
40
+ ## What gets better
41
+
42
+ With Waypoint, Codex should become better at:
43
+
44
+ - understanding the product and technical context before it starts
45
+ - planning work in enough detail to avoid avoidable mistakes
46
+ - writing code that matches the codebase and holds up in production
47
+ - staying on track during larger or longer-running tasks
48
+ - reviewing and verifying its own work before calling it done
49
+ - improving its own guidance when the user corrects it
50
+
51
+ ## Why Waypoint exists
52
+
53
+ Waypoint is for people using Codex on real apps and real codebases, not just
54
+ tiny one-off edits.
55
+
56
+ It exists because most Codex users should not have to manually remember every
57
+ best practice, every guardrail, every planning question, and every review step
58
+ for every task.
59
+
60
+ Waypoint packages that expertise into the repo so Codex starts from a much
61
+ better default.
62
+
63
+ ## What Waypoint adds
64
+
65
+ ### 1. Better default behavior
66
+
67
+ Waypoint gives Codex stronger repo guidance through the generated contract,
68
+ operating manual, core behavior files, and repo-local instructions.
69
+
70
+ That means the agent is pushed to:
71
+
72
+ - investigate before narrating status
73
+ - ask better questions about the product, architecture, and constraints
74
+ - explain what it found in a clear way
75
+ - verify what it changed
76
+ - leave the repo clearer than it found it
77
+
78
+ ### 2. Better planning
79
+
80
+ Waypoint ships a thorough planning workflow for work that should not start from
81
+ guesswork.
82
+
83
+ That workflow pushes the agent to:
15
84
 
16
- Waypoint adds:
85
+ - interview the user until the real requirements are clear
86
+ - produce a detailed plan before implementation
87
+ - challenge that plan with a reviewer agent
88
+ - tighten the plan before coding starts
17
89
 
18
- - explicit repo-local memory
19
- - strong default collaboration
20
- - optional structured workflows when the task actually needs them
90
+ The goal is simple: fewer assumptions, fewer surprises, and a much better shot
91
+ at one-shot execution.
21
92
 
22
- The default mode centers a simple loop:
93
+ ### 3. Better code quality
23
94
 
24
- - investigate the issue
25
- - explain what is happening
26
- - fix what you can
27
- - verify it
28
- - leave the repo clearer than you found it
95
+ Waypoint does not assume Codex will naturally write production-quality code by
96
+ default.
29
97
 
30
- ## Core idea
98
+ It adds guardrails that push the agent toward:
31
99
 
32
- Waypoint keeps the good parts of a repo operating system:
100
+ - stronger coding standards
101
+ - better fit with the existing codebase
102
+ - fewer lazy shortcuts
103
+ - fewer architecture mistakes
104
+ - fewer duplicated or premature abstractions
33
105
 
34
- - durable context in files
35
- - explicit startup and routing
106
+ Reviewer agents and audit workflows add another pass before merge when the work
107
+ needs it.
108
+
109
+ ### 4. Better end-to-end execution
110
+
111
+ Waypoint also helps Codex follow through on bigger tasks.
112
+
113
+ It includes workflows for:
114
+
115
+ - long-running task tracking
116
+ - ship-readiness audits
117
+ - merge-ready ownership
118
+ - deliberate review passes before PR or merge
119
+
120
+ This helps the agent keep moving until the work is actually ready, not just
121
+ "probably done."
122
+
123
+ ### 5. Self-improvement
124
+
125
+ Waypoint treats user corrections as product input, not just conversation noise.
126
+
127
+ When the user corrects behavior, rules, or workflow, the agent is pushed to
128
+ update the right durable files so the same issue is less likely to happen
129
+ again.
130
+
131
+ That includes:
132
+
133
+ - user-scoped guidance
134
+ - project-scoped guidance
36
135
  - repo-local skills
37
- - reusable reviewer agents
38
- - generated context for continuity
136
+ - retrospectives that turn friction from the current conversation into lasting
137
+ improvements
39
138
 
40
- Those systems work best when they stay explicit and well-scoped.
139
+ ### 6. Better continuity
41
140
 
42
- Structured workflows belong in tools:
141
+ Waypoint gives Codex explicit continuity artifacts so the next session does not
142
+ start half-blind.
43
143
 
44
- - review loops
45
- - ship-readiness passes
46
- - trackers
47
- - retrospectives
48
- - pre-PR hygiene
144
+ That includes:
49
145
 
50
- That keeps the default conversation focused on diagnosis, progress, and verification.
146
+ - a generated docs index that tells the agent which docs exist and when to read
147
+ them
148
+ - a live workspace file that records what is going on right now
149
+ - a generated recent thread file that carries the most important prior
150
+ conversation context forward
51
151
 
52
152
  ## What Waypoint sets up
53
153
 
@@ -57,32 +157,43 @@ Waypoint scaffolds a Codex-friendly repo around a few core pieces:
57
157
  - `.waypoint/WORKSPACE.md` for live operational state
58
158
  - `.waypoint/docs/` for long-lived project docs
59
159
  - `.waypoint/plans/` for durable plan documents
60
- - `.waypoint/DOCS_INDEX.md` for docs and plans routing
160
+ - `.waypoint/DOCS_INDEX.md` for docs and plans routing, so the agent knows what
161
+ to read and when
61
162
  - `.waypoint/context/` for generated startup context
62
- - `.waypoint/track/` for long-running work that truly needs durable progress tracking
163
+ - `.waypoint/context/RECENT_THREAD.md` for compact continuity from the previous
164
+ conversation
165
+ - `.waypoint/track/` for long-running work that truly needs durable progress
166
+ tracking
63
167
  - `.agents/skills/` for optional structured workflows
64
168
  - `.codex/` for optional reviewer and helper agents
65
169
 
66
- The philosophy is simple:
170
+ By default, Waypoint routes docs from `.waypoint/docs/` and plans from
171
+ `.waypoint/plans/`.
172
+ If your repo keeps routable docs elsewhere, you can add more explicit roots in
173
+ `.waypoint/config.toml` with `docs_dirs` and `plans_dirs`.
174
+ Waypoint scans each configured root recursively and only includes Markdown files
175
+ with valid Waypoint frontmatter.
67
176
 
68
- - less hidden runtime magic
69
- - more explicit repo-local state
70
- - stronger default collaboration
71
- - investigation before status narration
72
- - structured workflows that stay in their own tools
177
+ The continuity story matters:
73
178
 
74
- By default, Waypoint routes docs from `.waypoint/docs/` and plans from `.waypoint/plans/`.
75
- If your repo keeps routable docs elsewhere, you can add more explicit roots in `.waypoint/config.toml` with `docs_dirs` and `plans_dirs`.
76
- Waypoint scans each configured root recursively and only includes Markdown files with valid Waypoint frontmatter.
179
+ - `.waypoint/DOCS_INDEX.md` helps the agent find the right docs before work
180
+ - `.waypoint/WORKSPACE.md` helps the next session understand what is in flight
181
+ - `.waypoint/context/RECENT_THREAD.md` helps the agent retain the important
182
+ parts of the previous conversation
77
183
 
78
184
  ## Best fit
79
185
 
80
186
  Waypoint is most useful when you want:
81
187
 
82
- - multi-session continuity in a real repo
83
- - clear separation between cross-project user guidance and repo-specific guidance
84
- - a cleaner default collaboration style
85
- - optional planning, review, QA, and release workflows that travel with the project
188
+ - a better default Codex workflow in a real repo
189
+ - stronger planning before implementation starts
190
+ - stronger coding standards and review guardrails
191
+ - better follow-through on long tasks
192
+ - a personal workflow that can live in almost any repo without becoming a team
193
+ rollout
194
+
195
+ Waypoint is primarily an individual tool.
196
+ Most of its repo-local state is meant to stay personal and local by default.
86
197
 
87
198
  If you only use Codex for tiny one-off edits, Waypoint is probably unnecessary.
88
199
 
@@ -154,7 +265,37 @@ Waypoint ships a strong default skill pack for real coding work:
154
265
  These are repo-local, so the workflow travels with the project.
155
266
 
156
267
  The important design choice is that they stay out of the always-on voice.
157
- Each skill explains what it is for and when it should be invoked.
268
+ Each skill exists to improve the result when the task needs more rigor, without
269
+ turning every normal interaction into a heavy process.
270
+
271
+ ## How to get full value
272
+
273
+ Installing Waypoint improves Codex's defaults right away, but the full workflow
274
+ is not completely automatic.
275
+
276
+ Some of Waypoint's biggest advantages come from user-invoked skills that should
277
+ be used deliberately when the moment calls for them.
278
+
279
+ The most important ones are:
280
+
281
+ - `code-guide-audit` when you want a code quality pass against your repo's
282
+ standards and working rules
283
+ - `backend-ship-audit` when backend work needs a deeper production-readiness
284
+ pass
285
+ - `frontend-ship-audit` when frontend work needs a deeper product, UX, and ship
286
+ readiness pass
287
+ - `docs-sync` when the implementation changed and the repo docs should be
288
+ brought back in sync
289
+ - `conversation-retrospective` when the conversation exposed friction,
290
+ corrections, or workflow problems that should become durable improvements
291
+ - `merge-ready-owner` when you want the agent to own the task all the way to a
292
+ merge-ready result with stronger autonomy
293
+
294
+ The practical rule is:
295
+
296
+ - install Waypoint for better defaults
297
+ - invoke the higher-rigor skills when you want a stronger planning, audit,
298
+ review, docs, or closeout pass
158
299
 
159
300
  ## Reviewer agents
160
301
 
@@ -172,24 +313,13 @@ Waypoint is opinionated, but explicit:
172
313
 
173
314
  - state lives in files you can inspect
174
315
  - docs routing is generated, not guessed from memory
175
- - the default contract tells the agent to investigate first
316
+ - the default contract tells the agent to ask better questions and investigate
317
+ first
176
318
  - durable guidance is separated into user-scoped AGENTS, project-scoped AGENTS, live workspace state, project docs, and plan docs
177
319
  - visual explanation stays lightweight: Mermaid in chat and screenshots from real UI inspection
178
320
  - heavier workflows stay in optional skills
179
-
180
- ## Install
181
-
182
- Waypoint requires Node 20+.
183
-
184
- ```bash
185
- npm install -g waypoint-codex
186
- ```
187
-
188
- Or run it without a global install:
189
-
190
- ```bash
191
- npx waypoint-codex@latest --help
192
- ```
321
+ - user corrections are supposed to improve the system instead of disappearing
322
+ into chat history
193
323
 
194
324
  ## Main commands
195
325
 
package/package.json CHANGED
@@ -1,6 +1,6 @@
1
1
  {
2
2
  "name": "waypoint-codex",
3
- "version": "0.15.0",
3
+ "version": "0.16.0",
4
4
  "description": "Codex-native repository operating system: scaffolding, docs routing, repo-local skills, doctor, and sync.",
5
5
  "license": "MIT",
6
6
  "type": "module",
@@ -36,6 +36,11 @@ When the product surface makes it practical, extend done to include:
36
36
 
37
37
  Do not stop at "the code compiles" or "the first push is up."
38
38
 
39
+ During the loop, keep live execution state current:
40
+
41
+ - update `WORKSPACE.md` as milestones, blockers, verification state, and next steps change
42
+ - if a tracker exists or the work has become tracker-worthy, update the tracker during the work instead of reconstructing it later
43
+
39
44
  ## Step 1: Reconfirm The Scope And Ownership Mode
40
45
 
41
46
  - Make sure the plan is already approved or the user explicitly said to proceed.
@@ -64,7 +69,7 @@ For bugs, prefer reproducing the problem first, then fixing it, then proving the
64
69
 
65
70
  Use the repo's existing skills and reviewer agents instead of inventing a parallel process.
66
71
 
67
- - Use `work-tracker` when the work or verification checklist is too large for `WORKSPACE.md`.
72
+ - Use `work-tracker` early when the work becomes non-trivial, multi-step, review-heavy, or checklist-driven enough that `WORKSPACE.md` alone will stop being a good live record.
68
73
  - Use `docs-sync` when shipped behavior, routes, contracts, or commands changed materially.
69
74
  - Use `pre-pr-hygiene` before pushing or opening/updating a PR when the change surface is substantial.
70
75
  - Use `pr-review` once active PR review or automated review has started.
@@ -75,8 +80,10 @@ If the repo ships reviewer agents under `.codex/agents/`, use them in the closeo
75
80
  - run `code-reviewer` for every non-trivial implementation slice before declaring the work clear
76
81
  - run `code-health-reviewer` when the change is medium or large, especially when it adds structure, duplicates logic, or introduces new abstractions
77
82
  - launch them in parallel when both apply
83
+ - use them at meaningful milestones, not only at the very end: after substantial implementation chunks, before opening or materially updating a PR, after fixing substantial findings, and before final closeout
78
84
  - treat them as fresh closeout passes, not as optional decoration
79
- - if meaningful fixes follow from their findings, rerun the most relevant verification and, when warranted, rerun fresh reviewer passes instead of trusting stale results
85
+ - if either reviewer finds anything more serious than obvious optional polish, fix those findings, rerun the most relevant verification, and run fresh reviewer passes instead of trusting stale results
86
+ - keep iterating until the remaining reviewer feedback is only nitpicks or none
80
87
 
81
88
  If those reviewer agents are not present in the repo, do the equivalent closeout thinking locally and continue instead of blocking on missing helpers.
82
89
 
@@ -96,6 +103,7 @@ If an existing repo-local skill clearly matches the verification surface, use it
96
103
  - Run builds, lint, migrations, or focused smoke tests when they are part of the real risk surface.
97
104
  - Fix failing checks before pushing unless the user explicitly accepts an exception.
98
105
  - For user-facing flows, do at least one realistic manual or UI-driven pass beyond pure unit coverage.
106
+ - Update `WORKSPACE.md` and any active tracker with the current verification state before moving on.
99
107
 
100
108
  Do not push a branch that still obviously fails its own touched-surface checks.
101
109
 
@@ -110,6 +118,8 @@ When the repo uses PRs:
110
118
 
111
119
  If the repo does not use PRs, keep moving through the equivalent review and handoff workflow instead of forcing PR-shaped steps.
112
120
 
121
+ Before opening or materially updating the PR on non-trivial work, strongly prefer a fresh reviewer-agent pass when those agents are available.
122
+
113
123
  ## Step 8: Babysit The PR Instead Of Dropping It
114
124
 
115
125
  When the repo uses PRs, CI, or preview environments:
@@ -130,6 +140,7 @@ Once review starts:
130
140
  - fix valid findings
131
141
  - reply inline where the workflow supports inline reply
132
142
  - rerun the relevant verification after review-driven fixes
143
+ - if the fixes were meaningful, run fresh reviewer-agent passes before you call the work clear when those agents are available
133
144
 
134
145
  Do not leave comments unanswered just because the code changed.
135
146
 
@@ -161,12 +172,13 @@ Keep the handoff plain and direct. The point of this skill is to reduce the user
161
172
  ## Gotchas
162
173
 
163
174
  - Do not mistake planning approval for permission to stop at implementation; this skill owns the full closeout.
175
+ - Do not let `WORKSPACE.md` or an active tracker fall behind reality while the work is in flight.
164
176
  - Do not rely only on automated tests when the risky surface is interactive.
165
177
  - Do not let stale previews, staging selectors, old PR branches, or half-deployed environments quietly poison verification.
166
178
  - Do not treat CI failures, review comments, or rollout gates as outside the task once the user asked for full ownership.
167
179
  - Do not declare success while known meaningful review findings or failing checks still exist.
168
180
  - Do not confuse a reusable test harness or scripted UI test with the final walkthrough artifact; the artifact should show the real verified surface when practical.
169
- - Do not forget the reviewer-agent loop when `code-reviewer` and `code-health-reviewer` are available. They are part of the closeout signal, not an afterthought.
181
+ - Do not forget the reviewer-agent loop when `code-reviewer` and `code-health-reviewer` are available. They are part of the closeout signal, not an afterthought, and serious findings should reopen the fix-and-review cycle.
170
182
 
171
183
  ## Keep This Skill Sharp
172
184
 
@@ -1,11 +1,11 @@
1
1
  ---
2
2
  name: work-tracker
3
- description: Create or maintain a durable tracker under `.waypoint/track/` for large multi-step work. Use when implementation will span multiple sessions, when an audit or review produces many fix items, when verification has a long checklist, or whenever `WORKSPACE.md` would become too detailed if it tried to hold the whole execution log.
3
+ description: Create or maintain a durable tracker under `.waypoint/track/` for any non-trivial workstream that needs ongoing execution state. Use when work has multiple steps, meaningful verification, review follow-ups, milestone checkpoints, or any real chance that `WORKSPACE.md` alone will stop being enough as the work evolves.
4
4
  ---
5
5
 
6
6
  # Work Tracker
7
7
 
8
- Use this skill when the work is too large, too long-running, or too itemized to live safely in `WORKSPACE.md`.
8
+ Use this skill when the work has enough moving parts that the next state should not live only in chat or in a few workspace bullets.
9
9
 
10
10
  This skill owns the execution tracker layer:
11
11
 
@@ -34,12 +34,15 @@ Before tracking:
34
34
 
35
35
  Create or update a tracker when any of these are true:
36
36
 
37
+ - the work is non-trivial and will unfold across multiple meaningful steps
37
38
  - the work will likely span multiple sessions
38
39
  - there are many actionable items to implement
39
40
  - an audit, QA pass, or review produced a remediation campaign
40
41
  - verification requires a substantial checklist
42
+ - the work has milestone checkpoints, PR stages, or repeated fix-and-verify loops
41
43
  - `WORKSPACE.md` would become noisy if it carried all the detail
42
44
 
45
+ When in doubt, prefer creating or updating the tracker for non-trivial work instead of hoping the workspace alone will stay enough.
43
46
  Small, single-shot work does not need a tracker.
44
47
 
45
48
  ## Step 1: Choose The Tracker File
@@ -99,6 +102,7 @@ The tracker should answer "what exactly is happening across the whole workstream
99
102
  - Update `last_updated` whenever you materially change the tracker.
100
103
  - Mark completed items done instead of deleting the record.
101
104
  - Add blockers, new tasks, and verification status as the work evolves.
105
+ - Update the tracker during the work, not only at the end. If a milestone, blocker, review round, or verification result changed reality, the tracker should already reflect it.
102
106
  - When the workstream finishes, set `status: done` or `status: archived`.
103
107
 
104
108
  Do not let the tracker become fiction. It must match reality.
@@ -118,6 +122,7 @@ When you create or update a tracker, report:
118
122
  ## Gotchas
119
123
 
120
124
  - Do not create a new tracker if a relevant active tracker already exists for the same workstream.
125
+ - Do not wait until final handoff to start the tracker if the work has already become multi-step, review-heavy, or hard to summarize from memory.
121
126
  - Do not let the tracker become fiction. Completed items, blockers, and verification state should match reality.
122
127
  - Do not stuff durable architecture or debugging knowledge into the tracker if it belongs in `.waypoint/docs/`.
123
128
  - Do not leave `WORKSPACE.md` carrying the full execution log after a tracker exists.
@@ -57,8 +57,11 @@ If something important lives only in your head or in the chat transcript, the re
57
57
  - Before making meaningful frontend or backend decisions, inspect the available user-scoped and project-scoped `AGENTS.md` guidance. If the task depends on frontend or backend context that is missing from the project-scoped guidance and routed docs, use the corresponding `*-context-interview` skill to fill that gap instead of guessing.
58
58
  - Update the user-scoped `AGENTS.md` when you learn a durable preference, workflow rule, or default that should apply across projects and your environment allows you to edit it.
59
59
  - Update the project-scoped repo `AGENTS.md` when you learn durable repo truth, project constraints, or stable project-specific collaboration rules.
60
- - Update `.waypoint/WORKSPACE.md` as live execution state when progress meaningfully changes. In multi-topic sections, prefix new or materially revised bullets with a local timestamp like `[2026-03-06 20:10 PST]`.
61
- - For large multi-step work that is likely to span sessions or needs durable progress state, create or update a tracker in `.waypoint/track/`, keep detailed execution state there, and point at it from `## Active Trackers` in `.waypoint/WORKSPACE.md`.
60
+ - Treat `.waypoint/WORKSPACE.md` as mandatory live execution state, not end-of-task paperwork.
61
+ - Update `.waypoint/WORKSPACE.md` during the work whenever the active goal, phase, next step, blocker, verification state, or review state materially changes. In multi-topic sections, prefix new or materially revised bullets with a local timestamp like `[2026-03-06 20:10 PST]`.
62
+ - Do not wait until final handoff to update workspace state. If the work took enough effort that the next agent would benefit from a current snapshot, the workspace should already say so.
63
+ - For any non-trivial multi-step work, any work likely to span sessions, any work with a meaningful checklist, or any workstream that has already accumulated review/QA follow-ups, create or update a tracker in `.waypoint/track/`, keep detailed execution state there during the work, and point at it from `## Active Trackers` in `.waypoint/WORKSPACE.md`.
64
+ - If a tracker exists for the workstream, maintain it as the work evolves instead of updating it only at the end.
62
65
  - Update `.waypoint/docs/` when durable project knowledge changes, update `.waypoint/plans/` when durable plans change, and refresh each changed routable doc's `last_updated` field.
63
66
  - Rebuild `.waypoint/DOCS_INDEX.md` whenever routable docs change.
64
67
  - Rebuild `.waypoint/TRACKS_INDEX.md` whenever tracker files change.
@@ -66,9 +69,11 @@ If something important lives only in your head or in the chat transcript, the re
66
69
  - Let skills carry their own invocation guidance. The always-on contract should only keep the high-level rule: use repo-local skills deliberately when they help the current task.
67
70
  - When spawning `coding-agent`, default to `fork_context: false` and choose the model/reasoning pair that fits the slice. Use stronger models when the delegated slice is user-visible, architecturally important, or hard to unwind.
68
71
  - When spawning reviewer agents or other non-`coding-agent` subagents, explicitly set `fork_context: false` and choose the model/reasoning pair that matches the risk and importance of the second pass.
69
- - Use the repo-local skills and reviewer agents deliberately, not reflexively.
72
+ - Use the repo-local skills and reviewer agents deliberately, but do not underuse them on work that is expensive to get wrong.
73
+ - For non-trivial work, strongly prefer reviewer-agent passes between major implementation milestones, before opening or updating a PR, after fixing substantial findings, and before final closeout when the environment allows those agents to run.
70
74
  - If you created a PR earlier in the current session and need to push more work, first confirm that PR is still open. If it is closed, create a fresh branch from `origin/main` and open a fresh PR instead of pushing more commits to the old PR branch.
71
75
  - Treat reviewer agents as one-shot workers: once a reviewer returns findings, read the result and close it. If another review pass is needed later, spawn a fresh reviewer instead of reusing the same thread.
76
+ - If `code-reviewer` or `code-health-reviewer` surface anything more serious than optional polish, fix the findings, rerun the relevant verification, and launch fresh passes until the remaining feedback is only nitpicks or none.
72
77
  - Do not kill long-running subagents or reviewer agents just because they are slow.
73
78
  - When waiting on reviewers, subagents, CI, automated review, or external jobs that you deliberately chose to start, wait as long as required. There is no fixed timeout where waiting itself becomes the problem.
74
79
  - Never interrupt in-flight work just to force a partial result, salvage something quickly, or avoid making the user wait longer.
@@ -133,6 +138,7 @@ Deliberate closeout review is available when you want a second pass for ship-rea
133
138
  - If you use it, follow the skill's loop fully: define the reviewable slice, run the needed reviewers, wait for the outputs, fix meaningful findings, and rerun fresh passes when warranted.
134
139
  - Treat reviewer agents as one-shot workers. Once a reviewer returns its findings, read the result and close it.
135
140
  - Do not widen the scope casually; keep the second pass anchored to the slice you are actually trying to validate.
141
+ - For non-trivial work, the healthy default is to use reviewer passes at meaningful milestones instead of saving all second-pass scrutiny for the very end.
136
142
 
137
143
  ## Quality bar
138
144
 
@@ -94,13 +94,18 @@ Delivery expectations:
94
94
  - Only come back before that if you hit a genuine blocker you cannot clear with the codebase, tools, or available context. If that happens, say it plainly and be explicit about what remains unverified.
95
95
 
96
96
  Working rules:
97
- - Keep `.waypoint/WORKSPACE.md` current as the live execution state, with timestamped new or materially revised entries in multi-topic sections
97
+ - Treat `.waypoint/WORKSPACE.md` as a mandatory live execution log, not a closeout chore.
98
+ - Update `.waypoint/WORKSPACE.md` during the work whenever the active goal, current phase, next step, blocker, verification state, or handoff context materially changes.
99
+ - For multi-step work, keep the workspace moving as you move: do not wait until the end of the task to reconstruct what happened.
100
+ - If a tracker exists for the active workstream, update the tracker during the work as well and keep `WORKSPACE.md` pointing at the current tracker state.
98
101
  - Update user-scoped `AGENTS.md` when you learn a durable preference, standing rule, or default that should apply across projects and your environment allows you to edit that file
99
102
  - Update the project-scoped repo `AGENTS.md` when you learn durable repo truth, project constraints, or stable project-specific collaboration rules
100
103
  - Update `.waypoint/docs/` when durable project knowledge changes, update `.waypoint/plans/` when a durable plan changes, and refresh `last_updated` on touched routable docs
101
104
  - Keep most work in the main agent. Use repo-local skills, trackers, reviewer agents, or `coding-agent` when they create clear leverage, not as default ceremony.
102
105
  - Let repo-local skills describe their own triggers. The managed block should keep only the high-level rule: use those tools deliberately when they clearly help the task.
103
- - Use reviewer agents when an independent second pass would materially improve the result, not before every plan or fix by default.
106
+ - Use reviewer agents proactively at meaningful milestones when the work is non-trivial, risky, user-facing, merge-bound, or otherwise expensive to get wrong.
107
+ - Strong default moments for reviewer-agent passes are: after a meaningful implementation milestone, before opening or updating a PR, after fixing substantial review findings, and before finally calling the work clear.
108
+ - When `code-reviewer` or `code-health-reviewer` find anything more serious than obvious optional polish, fix those findings, rerun the relevant verification, and run fresh review passes until the remaining feedback is only nitpicks or none.
104
109
  - Treat `plan-reviewer`, `code-reviewer`, and `code-health-reviewer` as one-shot agents: once a reviewer returns findings, close it; if another pass is needed later, spawn a fresh reviewer instead of reusing the old thread
105
110
  - If you created a PR earlier in the current session and need to push more work, first confirm that PR is still open. If it is closed, create a fresh branch from `origin/main` and open a fresh PR instead of pushing more commits to the old PR branch
106
111
  - Treat the generated context bundle as required session bootstrap, not optional reference material