chore: stop format-gating generated trajectory artifacts - #1530
Conversation
`format:check` fails on main:
[warn] .agentworkforce/trajectories/task-1507/completed/2026-08/traj_esvqzlbnhqbt/summary.md
[warn] .agentworkforce/trajectories/task-1507/completed/2026-08/traj_esvqzlbnhqbt/trajectory.json
Those files were committed by #1520 at 2026-08-15 20:34 and are generated
agent runtime output, not source. Prettier had no ignore rule for them, so
every open PR now fails Prettier regardless of what it changed — relay#1529
touches one Rust file and fails on this.
Reformatting them would be wrong: they are regenerated on every run, so the
next trajectory reintroduces the failure. Ignore the directory instead,
alongside the other generated paths already listed.
Related: #1385 adds the matching .gitignore rule; this is the formatter half.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
Bugbot is not enabled for your account, so this pull request was not reviewed. Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Prettier ignore configuration now excludes ChangesPrettier ignore rules
Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: ⚪ Minimal · up to This is a narrowly scoped formatter-ignore change for generated artifacts, with no actionable merge-blocking risk remaining beyond normal checks and review. Possibly related PRs
Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
format:checkcurrently fails onmain, so every open PR fails Prettier regardless of what it changes:Those files were committed by #1520 at 2026-08-15 20:34 and are generated agent runtime output, not source.
.prettierignorehad no rule covering them.Reformatting them would be the wrong fix — they are regenerated on every run, so the next trajectory reintroduces the failure. Ignoring the directory matches how the file already treats
dist/,target/andpackages/config/src/cli-registry.generated.ts.Verified: with this rule,
prettier --checkon both named files reportsAll matched files use Prettier code style!Related: #1385 adds the matching
.gitignorerule; this is the formatter half. Unblocks #1529, which touches one Rust file and fails only on this.Not merged — Khaliq owns the gate.
🤖 Generated with Claude Code