Skip to content

Comments

Fixed AI agents missing previously downloaded cache.#2310

Merged
AlexSkrypnyk merged 1 commit intomainfrom
feature/improve-agentic-instructions
Feb 20, 2026
Merged

Fixed AI agents missing previously downloaded cache.#2310
AlexSkrypnyk merged 1 commit intomainfrom
feature/improve-agentic-instructions

Conversation

@AlexSkrypnyk
Copy link
Member

@AlexSkrypnyk AlexSkrypnyk commented Feb 20, 2026

Summary by CodeRabbit

  • Documentation
    • Updated development guidelines with new task initialization guidance and critical command usage rules.

@coderabbitai
Copy link

coderabbitai bot commented Feb 20, 2026

Walkthrough

Documentation update to AGENTS.md adding guidance for checking cached documentation, project conventions, and fetching external docs before task investigation. Additionally extends critical rules with specific Drupal command preferences for config exports and script execution.

Changes

Cohort / File(s) Summary
Documentation & Guidance
AGENTS.md
Added "Before Starting Any Task" guidance block with cached docs lookup, project docs reference, and external docs fetching instructions. Extended critical rules with preferences for ahoy drush php:script over php:eval and config export requirements via ahoy drush cex.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A rabbit hops through guidance new,
Cached docs and rules, both tried and true,
Before each task, a checklist bright,
Config exports done just right!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Title check ⚠️ Warning The title mentions 'missing cache' but the summary shows the changes add a guidance block to check cached docs and cache from docs source—improving the cache checking process, not fixing something that was previously missing. Revise the title to accurately reflect the actual changes, such as 'Add AI agent guidance for checking and caching documentation' or similar phrasing that describes what was added.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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 feature/improve-agentic-instructions

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@AGENTS.md`:
- Line 65: Update the rule that currently reads "Never modify `scripts/vortex/`
- use `scripts/custom/` for your scripts" to clarify scope: state that
downstream projects should never modify `scripts/vortex/` and must use
`scripts/custom/`, but for the template repository (e.g., drevops/vortex)
modifications to `scripts/vortex/` are expected and allowed for maintenance;
reference the `scripts/vortex/` and `scripts/custom/` paths and explicitly call
out "template repo (drevops/vortex)" so reviewers understand the exception.


## Critical Rules

- **Never modify** `scripts/vortex/` - use `scripts/custom/` for your scripts
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Clarify the scripts/vortex/ rule for the template repo.

In the template repository, modifying scripts/vortex/ is expected for maintenance. The current blanket prohibition is misleading here. Consider scoping it to downstream projects instead.

✏️ Suggested wording
-- **Never modify** `scripts/vortex/` - use `scripts/custom/` for your scripts
+- **Downstream projects:** do not modify `scripts/vortex/`; use `scripts/custom/` instead.  
+- **Template repo (this repo):** updates to `scripts/vortex/` are allowed for template maintenance.

Based on learnings: “The drevops/vortex repository is the template source itself. When reviewing PRs in drevops/vortex, modifications to scripts/vortex/ are expected and correct…”

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@AGENTS.md` at line 65, Update the rule that currently reads "Never modify
`scripts/vortex/` - use `scripts/custom/` for your scripts" to clarify scope:
state that downstream projects should never modify `scripts/vortex/` and must
use `scripts/custom/`, but for the template repository (e.g., drevops/vortex)
modifications to `scripts/vortex/` are expected and allowed for maintenance;
reference the `scripts/vortex/` and `scripts/custom/` paths and explicitly call
out "template repo (drevops/vortex)" so reviewers understand the exception.

@AlexSkrypnyk
Copy link
Member Author

Code Coverage Report:
  2026-02-20 01:48:22

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.65% (177/187)

@github-actions
Copy link

Code Coverage Report:
  2026-02-20 01:48:24

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.65% (177/187)

@AlexSkrypnyk
Copy link
Member Author

Code Coverage Report:
  2026-02-20 01:51:46

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.65% (177/187)

@AlexSkrypnyk
Copy link
Member Author

Code Coverage Report:
  2026-02-20 01:52:08

 Summary:
  Classes:  0.00% (0/1)
  Methods:  0.00% (0/2)
  Lines:   94.65% (177/187)

@AlexSkrypnyk AlexSkrypnyk merged commit a585ab4 into main Feb 20, 2026
25 of 26 checks passed
@AlexSkrypnyk AlexSkrypnyk deleted the feature/improve-agentic-instructions branch February 20, 2026 02:17
@github-project-automation github-project-automation bot moved this from BACKLOG to Release queue in Vortex Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Release queue

Development

Successfully merging this pull request may close these issues.

1 participant