fix(spec): format npm_globals_spec.sh to satisfy treefmt check

[shunkakinoki]

Changes

• Flatten ShellSpec DSL indentation in spec/npm_globals_spec.sh to match shfmt output

Technical Details

• The bun npm shim purge and postinstall recovery blocks added in fix(npm-globals): purge bun npm shim that breaks @railway/cli install #2022/fix(npm-globals): run postinstall for packages missing native binaries #2023/fix(npm-globals): install transitive native binaries dropped by bun #2027 were committed with indented ShellSpec DSL, which shfmt (via treefmt) flattens
• This broke all four Nix workflow jobs on main (nix-format, nix-test, nix-flake, nix-check) since they all gate on the checks.x86_64-linux.treefmt flake check with --fail-on-change

Testing

• nix fmt -- --clear-cache --fail-on-change passes (0 changed)
• shellspec spec/npm_globals_spec.sh: 61 examples, 0 failures

Generated with Claude Code by Fable 5

---

Summary by cubic

Formats spec/npm_globals_spec.sh to match shfmt output so checks.x86_64-linux.treefmt passes and Nix CI jobs run again.

• Bug Fixes
  • Flattened ShellSpec blocks in bun npm shim purge and postinstall recovery to match shfmt.
  • Unblocks Nix workflows gated by --fail-on-change (nix-format, nix-test, nix-flake, nix-check).
  • No logic changes; verified with nix fmt -- --clear-cache --fail-on-change and shellspec (61 examples, 0 failures).

^{Written for commit baf8105. Summary will update on new commits.}

[indent-zero]

PR Summary

Reformats spec/npm_globals_spec.sh so it matches the repo's shell formatter (shfmt -i 2 -s, wired through treefmt.toml), unblocking the CI treefmt check. Only whitespace changes — no functional impact on the Shellspec suite, which is keyword-driven and indentation-insensitive.

• Unindents the four It … End groups inside Describe 'bun npm shim purge' (added in fix(npm-globals): purge bun npm shim that breaks @railway/cli install #2022) to column 0.
• Unindents the four It … End groups inside Describe 'postinstall recovery' (added in fix(npm-globals): run postinstall for packages missing native binaries #2023) to column 0, bringing them in line with the rest of the file.

Issues

No issues found.

CI Checks

Waiting for CI checks...

[cursor]

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

[coderabbitai]

Warning

Review limit reached

@shunkakinoki, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 11 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 58196e66-5bf0-458e-9835-546e58cc119b

📥 Commits

Reviewing files that changed from the base of the PR and between 7d9f90f and baf8105.

📒 Files selected for processing (1)

• spec/npm_globals_spec.sh

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch claude/elastic-kalam-5d33fb

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[gemini-code-assist]

Code Review

This pull request modifies spec/npm_globals_spec.sh by removing the indentation (leading spaces) for the test cases within the 'bun npm shim purge' and 'postinstall recovery' describe blocks. There are no review comments, and I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[mesa-dot-dev]

Mesa Description

TL;DR

Flattened ShellSpec DSL indentation in spec/npm_globals_spec.sh to satisfy the treefmt check and fix broken Nix workflow jobs on the main branch.

What changed?

• spec/npm_globals_spec.sh: Flattened indentation of the ShellSpec DSL for the bun npm shim purge and postinstall recovery blocks to match the output expected by shfmt (via treefmt).

^{Description generated by Mesa. Update settings}

[mesa-dot-dev]

Performed full review of 7d9f90f...baf8105

Analysis

This PR is a formatting-only change with minimal architectural risk. However, it reveals systemic pipeline fragility: the CI/CD relies on strict formatting gates via treefmt that interact poorly with DSL-heavy shell syntax (ShellSpec), creating brittleness without sufficient contributor guidance. The pipeline lacks preventive controls (pre-commit hooks, scoped formatter configs, documented indentation conventions for DSLs) to catch these issues before they block all workflow jobs. While functionally safe to merge, the underlying architecture should be hardened to prevent future formatter churn in DSL-sensitive files.

Tip

Help

Slash Commands:

• /review - Request a full code review
• /review latest - Review only changes since the last review
• /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
• /help - Get help with Mesa commands and configuration options

^{0 files reviewed | 0 comments | Edit Agent Settings • Read Docs}

[cubic-dev-ai]

No issues found across 1 file

_{Re-trigger cubic}