Skip to content

style: fix Biome format in settings-store-usage.test.ts (unbreak main CI) - #1260

Closed
UncertaintyDeterminesYou4ndMe wants to merge 1 commit into
apache:mainfrom
UncertaintyDeterminesYou4ndMe:fix/biome-format-settings-store-usage
Closed

style: fix Biome format in settings-store-usage.test.ts (unbreak main CI)#1260
UncertaintyDeterminesYou4ndMe wants to merge 1 commit into
apache:mainfrom
UncertaintyDeterminesYou4ndMe:fix/biome-format-settings-store-usage

Conversation

@UncertaintyDeterminesYou4ndMe

Copy link
Copy Markdown
Contributor

The typecheck CI job has been failing on main since #1254 landed: the Biome format gate (#1122) reports one unformatted file, packages/storage/src/__tests__/settings-store-usage.test.ts (two tool_call object literals need rewrapping). Every open PR that merges with current main inherits the failure — e.g. #1253's latest run.

This is npx biome check --write output on that one file, formatting only, no behavior change. Verified npx biome check is clean on the file afterwards.

Failing run on main for reference: https://github.com/maka-agent/maka-agent/actions/runs/29712288307

main's typecheck CI job (Biome check --write=false) fails on this file
since apache#1254; rewrap the two tool_call literals the formatter expects.
Formatting only, no behavior change.

@Astro-Han Astro-Han left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No code findings. The target file now matches current main byte for byte, so this PR has no remaining net change. Closing it is cleaner than merging an empty change.

@Astro-Han Astro-Han closed this Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants