← Back to PRs

#22383: Chore: apply oxfmt baseline for CI check

by bmendonca3 open 2026-02-21 03:27 View on GitHub →
channel: msteams size: XS
## Summary - Problem: `pnpm check` currently fails on main due formatting drift in three files unrelated to feature/security changes. - Why it matters: unrelated PRs cannot reach green required checks. - What changed: applied `oxfmt --write` to the 3 offending files only. - What did NOT change (scope boundary): no logic/behavior changes. ## Change Type (select all) - [ ] Bug fix - [ ] Feature - [ ] Refactor - [ ] Docs - [ ] Security hardening - [x] Chore/infra ## Scope (select all touched areas) - [ ] Gateway / orchestration - [ ] Skills / tool execution - [ ] Auth / tokens - [ ] Memory / storage - [ ] Integrations - [ ] API / contracts - [ ] UI / DX - [x] CI/CD / infra ## Linked Issue/PR - Closes # - Related #22381 ## User-visible / Behavior Changes None. ## Security Impact (required) - New permissions/capabilities? (`No`) - Secrets/tokens handling changed? (`No`) - New/changed network calls? (`No`) - Command/tool execution surface changed? (`No`) - Data access scope changed? (`No`) ## Repro + Verification ### Steps 1. Run `pnpm check` on `main` baseline (fails formatting check). 2. Apply this PR. 3. Run `pnpm check` again. ### Expected - `pnpm check` succeeds. ### Actual - Locally verified: `pnpm check` passes. ## Evidence - [x] Failing test/log before + passing after - [ ] Trace/log snippets - [ ] Screenshot/recording - [ ] Perf numbers (if relevant) ## Human Verification (required) - Verified scenarios: - `pnpm check` passes locally after this formatting-only change. - Edge cases checked: - Confirmed only 3 files changed. - What you did **not** verify: - Any runtime behavior (none expected). ## Compatibility / Migration - Backward compatible? (`Yes`) - Config/env changes? (`No`) - Migration needed? (`No`) ## Failure Recovery (if this breaks) - How to disable/revert this change quickly: - Revert commit `303035a0a`. ## Risks and Mitigations None. <!-- greptile_comment --> <h3>Greptile Summary</h3> Applied oxfmt formatter to fix CI formatting checks. Changes are purely cosmetic - only import statement ordering was modified in 3 files (`src/agents/tools/common.ts`, `src/channels/plugins/actions/discord/handle-action.guild-admin.ts`, `src/discord/send.channels.ts`). No logic, behavior, or functionality changes. <h3>Confidence Score: 5/5</h3> - This PR is completely safe to merge - it contains only automated formatting changes - All changes are purely cosmetic import reordering performed by oxfmt formatter. No logic modifications, no new code, no behavioral changes. The PR description accurately describes the scope, and manual verification confirms only 3 files with 4 insertions and 4 deletions (net zero impact). This unblocks CI checks without any risk. - No files require special attention <sub>Last reviewed commit: 303035a</sub> <!-- greptile_other_comments_section --> <!-- /greptile_comment -->

Most Similar PRs