Skip to content

chore: tune analysis tools to non-blocking, organize repo files#44

Merged
ZeR020 merged 1 commit into
mainfrom
chore/tune-tools-organize-files
Jun 28, 2026
Merged

chore: tune analysis tools to non-blocking, organize repo files#44
ZeR020 merged 1 commit into
mainfrom
chore/tune-tools-organize-files

Conversation

@ZeR020

@ZeR020 ZeR020 commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Description

Tunes CodeRabbit, SonarCloud, and DeepSource to be helpful but not aggressive or blocking. Also organizes repo files — moves screenshots out of .github/ and cleans up dead .gitattributes rules.

Type of Change

  • Refactoring
  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Performance improvement

Changes

Tool tuning (non-blocking, less noisy)

CodeRabbit (.coderabbit.yaml):

  • Profile: assertivechill — fewer nitpicks, less verbose reviews
  • Added coverage/ to path filters
  • Added instructions: don't nitpick style (Prettier/DeepSource handle that), don't request tests for trivial one-liners
  • Disabled auto_reply in chat

SonarCloud (sonar-project.properties + check-sonarcloud-gate.sh):

  • Added secret-resolver.ts to coverage exclusions
  • Pre-push gate warning changed from alarming red ❌ to informational yellow ℹ️ (already non-blocking, exit 0)

DeepSource (.deepsource.toml):

  • Removed unnecessary react plugin (no React in this project — was causing false positives)

File organization

  • Moved .github/{banner,screenshot-*.png}docs/assets/ (485KB of images don't belong in .github/ — only templates and workflows live there)
  • Cleaned .gitattributes: removed dead graphify-out/ rules (directory doesn't exist), added dist/, coverage/, *.lock as linguist-generated
  • Updated .github/AGENTS.md and docs/AGENTS.md for moved assets

Checklist

  • I have run bun run typecheck and it passes
  • I have run bun run test and all tests pass
  • I have run bun run build and it succeeds
  • I have run bun run format:check and it passes
  • My code follows the project's style guidelines

Testing

Typecheck clean. No code changes — only config and file moves. Pre-commit and pre-push hooks passed.

Related Issues

N/A

Changelog Entry

Changed: CodeRabbit/SonarCloud/DeepSource tuned to non-blocking; screenshots moved to docs/assets/; .gitattributes cleaned

Contributor Credit

N/A

Tools:
- CodeRabbit: assertive → chill profile, add coverage/ to path_filters,
  instruct not to nitpick style (Prettier/DeepSource handle that) or
  request tests for trivial one-liners, disable auto_reply
- DeepSource: remove unnecessary react plugin (no React in project)
- SonarCloud: add secret-resolver.ts to coverage exclusions, make
  pre-push gate warning informational (yellow ℹ️) instead of alarming
  (red ❌) — already non-blocking (exit 0)
- check-sonarcloud-gate.sh: toned down failure output

Files:
- Move .github/{banner,screenshot-*.png} to docs/assets/ (485KB of
  images don't belong in .github/ — only templates and workflows live
  there)
- Clean .gitattributes: remove dead graphify-out/ rules (directory
  doesn't exist), add dist/coverage/lock as linguist-generated
- Update .github/AGENTS.md and docs/AGENTS.md for moved assets
Copilot AI review requested due to automatic review settings June 28, 2026 16:22
@coderabbitai

coderabbitai Bot commented Jun 28, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@ZeR020, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 39 minutes and 37 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

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 credits.

🚦 How do rate 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 see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 417bcb20-a1c2-48ac-91f4-d45fbd6e9f8e

📥 Commits

Reviewing files that changed from the base of the PR and between ec101d2 and c88ec7d.

⛔ Files ignored due to path filters (3)
  • docs/assets/banner.png is excluded by !**/*.png
  • docs/assets/screenshot-project-memory.png is excluded by !**/*.png
  • docs/assets/screenshot-user-profile.png is excluded by !**/*.png
📒 Files selected for processing (7)
  • .coderabbit.yaml
  • .deepsource.toml
  • .gitattributes
  • .github/AGENTS.md
  • docs/AGENTS.md
  • scripts/check-sonarcloud-gate.sh
  • sonar-project.properties
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/tune-tools-organize-files

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.

@deepsource-io

deepsource-io Bot commented Jun 28, 2026

Copy link
Copy Markdown

DeepSource Code Review

We reviewed changes in ec101d2...c88ec7d on this pull request. Below is the summary for the review, and you can see the individual issues we found as inline review comments.

See full review on DeepSource ↗

PR Report Card

Overall Grade   Security  

Reliability  

Complexity  

Hygiene  

Code Review Summary

Analyzer Status Updated (UTC) Details
JavaScript Jun 28, 2026 4:22p.m. Review ↗
Secrets Jun 28, 2026 4:22p.m. Review ↗

Important

AI Review is run only on demand for your team. We're only showing results of static analysis review right now. To trigger AI Review, comment @deepsourcebot review on this thread.

@sonarqubecloud

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adjusts repository analysis/review tooling to be informative and non-blocking, and reorganizes documentation assets to keep .github/ focused on GitHub-specific configuration.

Changes:

  • Tuned SonarCloud gate messaging to be explicitly non-blocking and expanded coverage exclusions.
  • Simplified DeepSource and CodeRabbit configuration to reduce noise.
  • Reorganized repo metadata: moved README assets into docs/assets/ and cleaned up .gitattributes linguist rules.

Reviewed changes

Copilot reviewed 7 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
sonar-project.properties Adds explanatory comments and updates sonar.coverage.exclusions (incl. secret-resolver.ts).
scripts/check-sonarcloud-gate.sh Changes failing-gate output to informational/non-blocking messaging and formatting.
docs/assets/banner.png Adds banner image under documentation assets.
docs/AGENTS.md Documents the new docs/assets/ location in the docs DOX index.
.github/AGENTS.md Updates .github/ DOX index to reflect README assets moved out.
.gitattributes Removes dead graphify-out/ rules and marks generated outputs for linguist.
.deepsource.toml Removes the unused React plugin from the JavaScript analyzer.
.coderabbit.yaml Switches to a less aggressive review profile and refines path filters/instructions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .gitattributes
Comment on lines +1 to +5
# Auto-generated files — excluded from GitHub linguist stats
*.lock linguist-generated=true
bun.lock linguist-generated=true
dist/** linguist-generated=true
coverage/** linguist-generated=true
@ZeR020 ZeR020 merged commit c1dc102 into main Jun 28, 2026
10 checks passed
@ZeR020 ZeR020 deleted the chore/tune-tools-organize-files branch June 28, 2026 16:25
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