chore(tooling): enforce formatting and refresh TypeScript checks#104239
Merged
Conversation
openclaw-barnacle
Bot
added
docs
openclaw-barnacle
Bot
added
extensions: meta
github-actions
Bot
added
the
dependencies-changed
Contributor
Dependency GuardThis PR changes dependency-related files. Maintainers should confirm these changes are intentional. Changed files:
Maintainer follow-up:
|
Contributor
Dependency graph changes notedThis PR includes dependency graph changes. The dependency guard is informational because the PR author is a repository admin or a member of
Security review is still recommended before merge when the dependency graph change is intentional. |
Contributor
|
Codex review: stale review; fresh review needed. Summary Next step |
openclaw-barnacle
Bot
added
extensions: diagnostics-otel
Contributor
Author
|
Merged via squash.
|
This was referenced Jul 11, 2026
chenyangjun-xy
pushed a commit
to chenyangjun-xy/openclaw
that referenced
this pull request
Jul 11, 2026
…nclaw#104239) * chore(tooling): enforce current formatter and refresh checks * chore(tooling): keep release changelog formatter-owned * chore(tooling): retain compatible Node type surface * ci: enforce formatting for docs-only changes * ci: isolate docs formatter check * chore(tooling): apply updated lint and format rules * chore(tooling): satisfy updated switch lint * style(ui): apply Linux formatter layout * test(doctor): match quiet local audio contribution * test(doctor): assert quiet output only * test(doctor): follow restored information contract
github-actions Bot
pushed a commit
to Desicool/openclaw
that referenced
this pull request
Jul 12, 2026
…nclaw#104239) * chore(tooling): enforce current formatter and refresh checks * chore(tooling): keep release changelog formatter-owned * chore(tooling): retain compatible Node type surface * ci: enforce formatting for docs-only changes * ci: isolate docs formatter check * chore(tooling): apply updated lint and format rules * chore(tooling): satisfy updated switch lint * style(ui): apply Linux formatter layout * test(doctor): match quiet local audio contribution * test(doctor): assert quiet output only * test(doctor): follow restored information contract
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What Problem This Solves
Resolves a tooling gap where repository formatting was only enforced by local hooks, allowing formatter drift to accumulate outside hook-enabled environments. It also updates the exact-pinned TypeScript-native, Oxlint, and Oxfmt toolchain.
Why This Change Was Made
The aggregate local check, CI lint shard, and docs-only CI path now run
format:check. The docs-only gate runs before the job clones the separate ClawHub repository, keeping the formatter scoped to OpenClaw. The change updates exact tool pins, removes the unusedScriptHostlibrary, applies the Oxfmt 0.58 baseline, and resolves the newly enabled Oxlint findings.The audit also confirmed that
scripts/**/*.tscannot be added honestly to the existing project graph as a small config-only change: release and E2E scripts import built artifacts, and the remaining source scripts expose substantial pre-existing type debt. This PR does not hide that debt with weakened compiler settings or exclusions.The attempted Node 22 type-definition pin was also rejected after exact CI proved that it does not model the repository's current crypto, filesystem, and test APIs. The existing Node 26 type surface remains until a dedicated Node-22 compatibility lane can be introduced without weakening project-wide checking.
User Impact
No runtime behavior change. Contributors and CI now get deterministic formatting enforcement, including docs-only changes, and the repository uses current exact-pinned TypeScript/Oxc tooling.
Evidence
git diff --checkclean.