Uh oh!
There was an error while loading. Please reload this page.
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 |
Uh oh!
There was an error while loading. Please reload this page.
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