Skip to content

task-observer: preflight updates from 2026-06-26 #67

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
divad12 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

task-observer: preflight updates from 2026-06-26 #67

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .agents/skills/preflight/SKILL.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -280,6 +280,8 @@ For each task, check whether the task has a `write failing test` + `verify test

The plan file is never modified. `/fly` instructs implementers to honor both plan steps and `[INJECTED]` checklist-only steps.

For tasks touching data model types, external API shapes, or cache key structures (Prisma models, query-result shapes, cache keys), inject an additional verification step before the first test: `[INJECTED] Read and verify actual <schema/API> field names against plan assumptions`. Plan-authored code stubs often carry wrong assumptions about field names, key structures, and return shapes; verifying the source file before writing tests is cheaper than discovering the mismatch mid-implementation.

### Manual-test convertibility analysis

For each phase, sweep ALL manual verification work the plan calls out. Sources:
Expand Down