Skip to content

fix: untrack AGENTS.md files — restore original .gitignore intent#46

Merged
ZeR020 merged 1 commit into
mainfrom
fix/untrack-agents-md
Jun 28, 2026
Merged

fix: untrack AGENTS.md files — restore original .gitignore intent#46
ZeR020 merged 1 commit into
mainfrom
fix/untrack-agents-md

Conversation

@ZeR020

@ZeR020 ZeR020 commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Description

Reverts PR #38's incorrect tracking of AGENTS.md files. The .gitignore was intentionally ignoring AGENTS.md — the DOX hierarchy is local-only and should not be committed.

Changes

  • Removed !AGENTS.md exception from .gitignore — restored original ignore rule
  • git rm --cached on all 14 AGENTS.md files — untracked from git (files stay on disk for local agent use)
  • No files deleted from disk — only removed from git tracking

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

Related Issues

Reverts the AGENTS.md tracking from PR #38

Changelog Entry

Fixed: untracked AGENTS.md files from git — restored original .gitignore intent (DOX hierarchy is local-only)

The .gitignore was intentionally ignoring AGENTS.md (DOX hierarchy is
local-only). PR #38 incorrectly added a !AGENTS.md exception and
force-added all 14 AGENTS.md files. This reverts that: removes the
exception, untracks all AGENTS.md files from git (files stay on disk
for local agent use).
Copilot AI review requested due to automatic review settings June 28, 2026 16:42
@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 19 minutes and 11 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: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a5f277ef-d740-438d-ac53-bb77977411d5

📥 Commits

Reviewing files that changed from the base of the PR and between e786480 and 9788df4.

📒 Files selected for processing (14)
  • .github/AGENTS.md
  • .husky/AGENTS.md
  • AGENTS.md
  • docs/AGENTS.md
  • scripts/AGENTS.md
  • src/AGENTS.md
  • src/services/AGENTS.md
  • src/services/ai/AGENTS.md
  • src/services/ai/providers/AGENTS.md
  • src/services/sqlite/AGENTS.md
  • src/services/user-profile/AGENTS.md
  • src/services/vector-backends/AGENTS.md
  • src/web/AGENTS.md
  • tests/AGENTS.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/untrack-agents-md

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 e786480...9788df4 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:42p.m. Review ↗
Secrets Jun 28, 2026 4:42p.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

@ZeR020 ZeR020 merged commit 39596e3 into main Jun 28, 2026
10 checks passed
@ZeR020 ZeR020 deleted the fix/untrack-agents-md branch June 28, 2026 16:44

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

This PR removes all AGENTS.md files from git tracking to revert the prior change that version-controlled the DOX instruction hierarchy, with the goal of keeping these files local-only.

Changes:

  • Removed (untracked) 14 AGENTS.md files across the repo (root, src/, src/services/, tests/, docs/, .github/, .husky/, etc.).
  • Eliminated the repository’s DOX hierarchy documentation from version control (files no longer exist in the repo after this change).

Reviewed changes

Copilot reviewed 14 out of 14 changed files in this pull request and generated no comments.

Show a summary per file
File Description
tests/AGENTS.md Removed test-suite DOX guidance from version control
src/web/AGENTS.md Removed web UI subtree DOX guidance from version control
src/services/vector-backends/AGENTS.md Removed vector backend DOX guidance from version control
src/services/user-profile/AGENTS.md Removed user profile DOX guidance from version control
src/services/sqlite/AGENTS.md Removed SQLite layer DOX guidance from version control
src/services/ai/providers/AGENTS.md Removed AI providers DOX guidance from version control
src/services/ai/AGENTS.md Removed AI subsystem DOX guidance from version control
src/services/AGENTS.md Removed services layer DOX guidance from version control
src/AGENTS.md Removed source-root DOX guidance from version control
scripts/AGENTS.md Removed scripts DOX guidance from version control
docs/AGENTS.md Removed docs subtree DOX guidance from version control
AGENTS.md Removed root DOX/contracts document from version control
.husky/AGENTS.md Removed Husky hooks DOX guidance from version control
.github/AGENTS.md Removed GitHub workflows/templates DOX guidance from version control
Comments suppressed due to low confidence (1)

AGENTS.md:1

  • PR intent is to ignore/untrack AGENTS.md, but the current .gitignore still contains a !AGENTS.md exception (and a comment indicating AGENTS.md should be tracked). That exception un-ignores AGENTS.md and makes it easy to accidentally re-add these files to git, which contradicts the PR’s stated goal.

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

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9788df485c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread AGENTS.md
@@ -1,145 +0,0 @@
# DOX framework

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Restore the AGENTS.md ignore rule before untracking

When these files are dropped from the index, the repository still leaves .gitignore with AGENTS.md immediately followed by !AGENTS.md, so Git does not ignore new AGENTS.md files (git check-ignore -q AGENTS.md exits 1; git check-ignore -h documents -q as suppressing output while checking paths). In this state the same files remain addable and can be reintroduced by a normal git add ., which undermines the stated fix to keep the DOX hierarchy local-only; remove the negation in the same change before untracking them.

Useful? React with 👍 / 👎.

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