docs(#6113): surface org-mode deprecation in agent-visible files - #6114
Conversation
|
🤖 Finished Review · ✅ Success · Started 6:54 PM UTC · Completed 7:06 PM UTC Commit: |
Site previewPreview: https://727d31e6-site.fullsend-ai.workers.dev Commit: |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
ReviewFindingsMedium
Low
Previous runReviewFindingsMedium
Low
Labels: PR modifies documentation files (AGENTS.md, docs/agents/triage.md) to surface org-mode deprecation. |
![fullsend-ai-review[bot]](https://avatars.githubusercontent.com/in/3478501?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
fullsend-ai-review
Bot
added
requires-manual-review
AGENTS.md had no mention of the per-org installation mode deprecation (ADR 0044), so agents generated and reviewed org-mode content as if it were active architecture. Add a bullet to the "How to work in this repo" section instructing agents not to add org-mode-specific content and to flag existing org-mode references as deprecated. Also annotate the org-mode reference in docs/agents/triage.md (line 73) with an inline deprecation note and link to ADR 0044, so the triage agent documentation reflects the current state. Note: pre-commit could not run in-sandbox (network policy blocks git fetch for hook environment setup). The post-script runs pre-commit authoritatively on the runner. Closes #6113
9d885b0 to
1233235
Compare
|
🤖 Finished Review · ✅ Success · Started 6:33 PM UTC · Completed 6:50 PM UTC Commit: |
1233235 to
061b455
Compare
|
🤖 Review · Commit: |
061b455 to
9ec3091
Compare
|
🤖 Review · Commit: |
The original bullet said per-org mode is deprecated but didn't call out that the rule applies to human contributors as well as agents. waynesun09 noted on #6113 that the generic phrasing already failed to prevent recurrence on PR #6375, so make the instruction explicit: don't add or extend org-mode-specific content, and flag it as deprecated rather than active architecture when reviewing. Deliberately don't link a specific tracking issue for the command removal inventory: #2302 (originally linked) closed and split into #6390-#6392, and #6391 (the deprecation-warning issue that enumerated the commands) closed as not-planned when the team decided to remove per-org support directly instead of warning first. Issue state churns too fast for a rule file to track — the ADR link is stable and durable in a way a specific issue number is not, so point only to ADR 0044 and let it be the single source of truth for removal scope. Apply the same clarification to the docs/agents/triage.md annotation so it reads consistently with AGENTS.md. Signed-off-by: Wayne Sun <gsun@redhat.com> Assisted-by: Claude
9ec3091 to
932a676
Compare
|
🤖 Finished Review · ✅ Success · Started 7:30 PM UTC · Completed 7:46 PM UTC Commit: |
|
🤖 Finished Retro · ❌ Failure · Started 7:42 PM UTC · Completed 7:43 PM UTC Commit: |
|
Review skipped — this PR is already merged. The Posted by fullsend post-review check |
1 participant
Summary
Surfaces the per-org installation mode deprecation (ADR 0044) in files that agents routinely read, so they avoid generating or engaging with org-mode content as active architecture.
Related Issue
Changes
Testing
grep -q 'ADR.0044' AGENTS.md)grep -n 'org.mode' docs/agents/triage.md | grep -qi 'deprecated')Note:
make lintcould not run in-sandbox (network policy blocks pre-commit hook environment setup). The post-script runs pre-commit authoritatively on the runner.Checklist
!for breaking changes)Closes #6113
Post-script verification
agent/6113-surface-org-mode-deprecation)65ceb961020290e43b008daa380765b44f35c5df..HEAD)