Conversation
Document that plan/design umbrella tasks should remain manual_only instead of getting a synthetic spec_ref, and point triage toward using plan_ref plus derived spec-backed tasks. Task: @01KJGRPX87E756YE1VFKXCWEZ3 Co-Authored-By: OpenAI Codex <codex@openai.com>
Add a phased plan artifact for reducing completeness warning debt, using the current validate baseline and explicit per-wave acceptance gates. Task: @01KJGRQ1BBMQ4VHWB8SSKG16RA Co-Authored-By: OpenAI Codex <codex@openai.com>
chapel
left a comment
chapel
left a comment
There was a problem hiding this comment.
Review Summary
Task: @01KJGRQ1BBMQ4VHWB8SSKG16RA
Spec: N/A (spike / plan task)
Plan Review
- Durable plan artifact exists at
docs/plans/completeness-warning-debt-reduction.md -
kspec plan get @plan-completeness-warning-debt-reduction --jsonmatches the committed document -
kspec validate --completeness --json --warnings-okstill supports the plan's stated baseline and warning-family framing
Regression
-
npm testpassed locally on this PR head (184files,4038tests;3skipped) - GitHub checks are green on the current PR head
Findings
This PR is blocked because it is conflated with unrelated work from other tasks. origin/main..docs/completeness-warning-debt-plan contains five commits, but only the tip commit (90504f101) belongs to this task. The PR currently includes CI/E2E and automation-skill changes from other task branches, and those unrelated CI changes are already failing on this PR (e2e (playwright chromium 1/3) and 2/3). Split this task onto a clean branch/PR that contains only the plan artifact (and any task-local docs references) before resubmitting.
Quality Gates
- Task context reviewed
- Plan content reviewed
- Local regression suite passed
- PR isolation / branch hygiene
- Remote CI clean on current HEAD
| kspec --version | ||
| kspec --help | ||
|
|
||
| e2e: |
There was a problem hiding this comment.
MUST-FIX: This task is a plan-only spike, but the PR is also carrying the E2E CI workflow work from other tasks. Keep this PR isolated to the completeness-plan artifact; the unrelated CI changes are what introduced the failing e2e checks on this review branch.
| - Mark `manual_only` — spikes are inherently human work | ||
| - `kspec task set @ref --automation manual_only --reason "Spike - output is knowledge"` | ||
|
|
||
| **If it is a plan/design umbrella task with derived child work:** |
There was a problem hiding this comment.
MUST-FIX: This automation-triage guidance change belongs to a different task (docs: clarify automation spec_ref exceptions) and should not be merged through the completeness-plan PR. Rebase or recreate the branch so this task only ships its own docs/plan changes.
1 participant
Summary
Validation
Task: @01KJGRQ1BBMQ4VHWB8SSKG16RA