#5884: fix: curly lint errors and formal conformance fork permissions
agents
Cluster:
GitHub Actions CI Improvements
## Summary
- Adds curly braces to two single-line `if` statements that violate the recently enabled `curly` lint rule
- `src/routing/resolve-route.ts:224` — parentPeerMatch early return
- `src/agents/system-prompt.ts:61` — userTimezone guard clause
- Wraps the `formal_conformance` workflow PR comment in a try/catch so it doesn't fail on fork PRs that lack write permissions
Both lint errors are currently failing on `main`. The formal conformance failure affects all fork PRs.
## Test plan
- [ ] `pnpm lint` passes
- [ ] `pnpm test` passes (no behavior change)
- [ ] `formal_conformance` job no longer fails on fork PRs
🤖 Generated with [Claude Code](https://claude.com/claude-code)
<!-- greptile_comment -->
<h2>Greptile Overview</h2>
<h3>Greptile Summary</h3>
This PR makes two small TypeScript changes to satisfy the newly enabled `curly` ESLint rule by adding braces around single-line `if` early returns in `src/routing/resolve-route.ts` and `src/agents/system-prompt.ts`.
It also updates the `formal_conformance` GitHub Actions workflow to avoid failing on fork PRs without write permissions by wrapping the `github.rest.issues.createComment` call in a `try/catch` and logging a warning when commenting is not permitted. This keeps the job informational while still surfacing drift via uploaded artifacts and the final step summary.
<h3>Confidence Score: 5/5</h3>
- This PR is safe to merge with minimal risk.
- Changes are narrowly scoped (curly-brace lint fixes and a workflow try/catch) and do not affect runtime logic beyond formatting; the workflow change only prevents non-critical permission failures on fork PRs.
- No files require special attention
<!-- greptile_other_comments_section -->
<sub>(4/5) You can add custom instructions or style guidelines for the agent [here](https://app.greptile.com/review/github)!</sub>
**Context used:**
- Context from `dashboard` - CLAUDE.md ([source](https://app.greptile.com/review/custom-context?memory=fd949e91-5c3a-4ab5-90a1-cbe184fd6ce8))
- Context from `dashboard` - AGENTS.md ([source](https://app.greptile.com/review/custom-context?memory=0d0c8278-ef8e-4d6c-ab21-f5527e322f13))
<!-- /greptile_comment -->
Most Similar PRs
#17426: ci(formal): don't fail on fork PRs when PR comment is blocked
by mitre88 · 2026-02-15
81.4%
#17621: fix: formal_conformance comment step should not fail job on fork PRs
by cmfinlan · 2026-02-16
78.5%
#7085: test: skip flaky workspace-paths & safe-bins tests on non-Linux/CI ...
by ThinkIbrokeIt · 2026-02-02
77.0%
#15852: fix: pass agentId when resolving IRC session paths
by MisterGuy420 · 2026-02-14
76.6%
#17452: ci: Grant write perms for Issues for formal-conformance.yml
by thesomewhatyou · 2026-02-15
75.7%
#6110: refactor: simplify boolean logic using nullish coalescing
by T1mn · 2026-02-01
75.6%
#11549: lint: add no-console rule and migrate 5 files to structured logger
by vaibhavtupe · 2026-02-08
75.2%
#6888: feat: PowerShell completion install/uninstall + templates script + ...
by ThinkIbrokeIt · 2026-02-02
75.1%
#11048: fix: address repository issues (env, author, CI comments, security ...
by cavula · 2026-02-07
75.1%
#21027: fix(tests): resolve TypeScript type errors in CLI test files
by kittipond2365 · 2026-02-19
74.7%