Skip to content

fix: remove project-level statusLine config#276

Merged
FSM1 merged 1 commit into
mainfrom
fix/statusline-config
Mar 6, 2026
Merged

fix: remove project-level statusLine config#276
FSM1 merged 1 commit into
mainfrom
fix/statusline-config

Conversation

@FSM1

@FSM1 FSM1 commented Mar 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • Remove statusLine from project .claude/settings.json — it belongs in global config only
  • The GSD installer incorrectly added it to project settings during the v1.22.4 update

Also restored the original statusline.js to ~/.claude/hooks/ globally (not tracked in repo).

Test plan

  • Verify statusline displays correctly after restart

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Simplified configuration settings by removing unused setup components.

StatusLine should be configured globally, not per-project.
The GSD installer incorrectly added it to project settings.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Entire-Checkpoint: 34f4d0df1d15
@coderabbitai

coderabbitai Bot commented Mar 6, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 30b3c844-7a40-4537-adf7-14faa508f2a8

📥 Commits

Reviewing files that changed from the base of the PR and between 8085eeb and 800a5de.

📒 Files selected for processing (1)
  • .claude/settings.json
💤 Files with no reviewable changes (1)
  • .claude/settings.json

Walkthrough

Removed the statusLine configuration entry from root settings, eliminating the reference to the status line command hook (gsd-statusline.js).

Changes

Cohort / File(s) Summary
Configuration Removal
.claude/settings.json
Removed statusLine hook configuration; permissions.deny entry remains unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and specifically describes the main change: removing project-level statusLine configuration from .claude/settings.json.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/statusline-config

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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

Comment @coderabbitai help to get the list of available commands and usage tips.

@FSM1 FSM1 merged commit a6878a4 into main Mar 6, 2026
15 checks passed
@FSM1 FSM1 deleted the fix/statusline-config branch March 6, 2026 22:47
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.

1 participant