Skip to content

chore(format): align oxfmt local/CI behavior#12579

Merged
vincentkoc merged 3 commits intoopenclaw:mainfrom
vincentkoc:vincentkoc-code/hooks-format-ci
Feb 19, 2026
Merged

chore(format): align oxfmt local/CI behavior#12579
vincentkoc merged 3 commits intoopenclaw:mainfrom
vincentkoc:vincentkoc-code/hooks-format-ci

Conversation

@vincentkoc
Copy link
Contributor

@vincentkoc vincentkoc commented Feb 9, 2026

Why

Formatter behavior can differ between local and CI runs, which causes avoidable red CI and slows review throughput. This PR makes formatter behavior explicit and adds a quick local diff workflow so contributors can reproduce CI formatting outcomes immediately.

Closes #9836
Note: lobster-biscuit

Split Context

This PR was split from closed umbrella PR #9761: #9761.

Detailed Changes

  • Updated .oxfmtrc.jsonc:
    • Added tabWidth: 2
    • Added useTabs: false
    • Added docker-compose.yml to ignorePatterns
  • Updated package.json scripts:
    • Added format:diff as oxfmt --write && git --no-pager diff

Related Links and Issues

Greptile Overview

Greptile Summary

This PR makes oxfmt formatting behavior consistent between local and CI by explicitly setting tabWidth: 2 and useTabs: false in .oxfmtrc.jsonc, and by ignoring docker-compose.yml.

It also adds a developer convenience script format:diff that runs oxfmt --write and immediately shows the resulting git diff, helping contributors reproduce CI formatting outcomes locally.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk.
  • The changes are limited to formatter configuration and a new local-only formatting helper script, with no impact on runtime code paths. The edits are syntactically valid JSONC/package.json changes and are unlikely to break CI behavior.
  • No files require special attention

Context used:

  • Context from dashboard - CLAUDE.md (source)
  • Context from dashboard - AGENTS.md (source)

@vincentkoc vincentkoc marked this pull request as ready for review February 9, 2026 10:34
@vincentkoc
Copy link
Contributor Author

@greptileai can you review please if its all good to go then say so. Small change

@vincentkoc
Copy link
Contributor Author

Rebased branch for merge conflicts, ready to merge again.

@vincentkoc
Copy link
Contributor Author

vincentkoc commented Feb 18, 2026

Just a small note for Barnacle and reviewers, this is still an open issue!
Rationale: We are currently blocking agents from creating good code as formatting depending on your Env will not work so you end up with large PRs and erroenous bad linting.

@vincentkoc vincentkoc merged commit f38e1a8 into openclaw:main Feb 19, 2026
23 checks passed
@vincentkoc vincentkoc deleted the vincentkoc-code/hooks-format-ci branch February 19, 2026 09:31
anschmieg pushed a commit to anschmieg/openclaw that referenced this pull request Feb 19, 2026
* chore(format): align oxfmt local/CI behavior
yneth-ray-openclaw pushed a commit to yneth-ray-openclaw/openclaw that referenced this pull request Feb 19, 2026
* chore(format): align oxfmt local/CI behavior
vignesh07 pushed a commit to pahdo/openclaw that referenced this pull request Feb 20, 2026
* chore(format): align oxfmt local/CI behavior
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: oxmft on github ci/cd not matching local

1 participant

Comments