Skip to content

GH#978: expand AGENTS.md to address recurring contributor error patterns#979

Merged
superdav42 merged 1 commit intomainfrom
feature/auto-20260429-011509-gh978
Apr 29, 2026
Merged

GH#978: expand AGENTS.md to address recurring contributor error patterns#979
superdav42 merged 1 commit intomainfrom
feature/auto-20260429-011509-gh978

Conversation

@superdav42
Copy link
Copy Markdown
Collaborator

@superdav42 superdav42 commented Apr 29, 2026

Summary

Triage of contributor-insight issue #978 (4 recurring error patterns from superdav42).

Worker triage assessment: Premise verified — error patterns are real. Outcome B: AGENTS.md already covers the four patterns comprehensively, but the repo file lagged behind the canonical target state. This PR fills the documented gaps.

Changes

AGENTS.md — Project Structure:

  • Added three missing test directories: tests/Builders/, tests/functional/, tests/unit/
  • These subdirectories exist in the repo but were absent from the project map, causing read:file_not_found errors when contributors searched for test files

AGENTS.md — Non-existent paths table:

  • Added inc/bootstrap.php — commonly attempted; plugin entry point is ultimate-multisite.php
  • Added tests/WP_Ultimo/bootstrap.php — commonly attempted; actual bootstrap is tests/bootstrap.php
  • Added inc/class-plugin.php / inc/class-main.php — common WP plugin naming convention that doesn't apply here; main class is inc/class-wp-ultimo.php

Error patterns addressed

Pattern Count Fix
read:file_not_found 38x 3 new not-exist table entries + full test structure documented
bash:other 75x Clearer test structure reduces navigation mistakes
edit:not_read_first 46x Already covered in AGENTS.md; no new gap found
webfetch:other 39x Already covered in AGENTS.md; no new gap found

Testing

Documentation-only change — no PHP/JS code modified.

git ls-files 'tests/Builders/' | head -1         # non-empty confirms directory exists
git ls-files 'inc/bootstrap.php'                  # empty confirms path correctly non-existent
git ls-files 'tests/WP_Ultimo/bootstrap.php'      # empty confirms path correctly non-existent

Resolves #978


aidevops.sh v3.13.10 plugin for OpenCode v1.3.17 with claude-sonnet-4-6 spent 6m and 17,830 tokens on this as a headless worker.

Summary by CodeRabbit

  • Documentation
    • Updated repository documentation to clarify test organization structure, including test suite layout and key file location references.

…(GH#978)

- Add missing test directories (Builders/, functional/, unit/) to project structure
- Clarify WP_Ultimo/ as main test suite in project map
- Add three commonly-attempted non-existent paths to the file-not-found table:
  inc/bootstrap.php, tests/WP_Ultimo/bootstrap.php, inc/class-plugin.php/class-main.php

Addresses contributor-insight patterns: read:file_not_found (38x) and
bash:other (75x) from superdav42 sessions. Fills gaps identified vs
canonical AGENTS.md target state.

Fixes #978
@superdav42 superdav42 added the origin:worker Auto-created by pulse labelless backfill (t2112) label Apr 29, 2026
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 35e161c1-b656-4f98-8df0-c6420974b1ac

📥 Commits

Reviewing files that changed from the base of the PR and between da4e600 and ac24fe0.

📒 Files selected for processing (1)
  • AGENTS.md

📝 Walkthrough

Walkthrough

This pull request updates AGENTS.md to document a reorganized test structure, establishing tests/WP_Ultimo/ as the main test suite alongside three additional subdirectories (Builders/, functional/, unit/), and adds clarification rows identifying plugin bootstrap (ultimate-multisite.php), test bootstrap (tests/bootstrap.php), and main plugin class file (inc/class-wp-ultimo.php) locations.

Changes

Cohort / File(s) Summary
Documentation Updates
AGENTS.md
Reorganizes test directory structure documentation and adds file-discovery guidance for plugin/test bootstrap locations and main plugin class file paths.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 Hops through the docs with glee,
Test directories now organized so neat,
Paths clarified for all to see,
Bootstrap files—complete!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately references issue #978 and clearly summarizes the main change: expanding AGENTS.md documentation to address contributor error patterns.
Linked Issues check ✅ Passed The PR successfully addresses all coding-related objectives from issue #978 by documenting missing test directories, clarifying non-existent paths, and providing explicit guidance on actual repository entry points.
Out of Scope Changes check ✅ Passed All changes are directly scoped to AGENTS.md documentation updates required to address the four recurring contributor error patterns identified in issue #978.
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
  • Commit unit tests in branch feature/auto-20260429-011509-gh978

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
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

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

@github-actions
Copy link
Copy Markdown

🔨 Build Complete - Ready for Testing!

📦 Download Build Artifact (Recommended)

Download the zip build, upload to WordPress and test:

🌐 Test in WordPress Playground (Very Experimental)

Click the link below to instantly test this PR in your browser - no installation needed!
Playground support for multisite is very limitied, hopefully it will get better in the future.

🚀 Launch in Playground

Login credentials: admin / password

@github-actions
Copy link
Copy Markdown

Performance Test Results

Performance test results for 8a82b40 are in 🛎️!

Note: the numbers in parentheses show the difference to the previous (baseline) test run. Differences below 2% or 0.5 in absolute values are not shown.

URL: /

Run DB Queries Memory Before Template Template WP Total LCP TTFB LCP - TTFB
0 41 (+2 / +4% ) 37.78 MB 884.50 ms 179.50 ms (+17.50 ms / +10% ) 1153.00 ms (+92.50 ms / +8% ) 2162.00 ms (+100.00 ms / +5% ) 2077.05 ms (+105.35 ms / +5% ) 90.10 ms (-3.30 ms / -4% )
1 56 49.03 MB 964.50 ms 148.50 ms 1115.50 ms 2108.00 ms 2025.55 ms 83.35 ms

@superdav42 superdav42 merged commit f165508 into main Apr 29, 2026
11 checks passed
@superdav42
Copy link
Copy Markdown
Collaborator Author

Summary

Triage of contributor-insight issue #978 (4 recurring error patterns from superdav42).
Worker triage assessment: Premise verified — error patterns are real. Outcome B: AGENTS.md already covers the four patterns comprehensively, but the repo file lagged behind the canonical target state. This PR fills the documented gaps.

Changes

AGENTS.md — Project Structure:

  • Added three missing test directories: tests/Builders/, tests/functional/, tests/unit/
  • These subdirectories exist in the repo but were absent from the project map, causing read:file_not_found errors when contributors searched for test files
    AGENTS.md — Non-existent paths table:
  • Added inc/bootstrap.php — commonly attempted; plugin entry point is ultimate-multisite.php
  • Added tests/WP_Ultimo/bootstrap.php — commonly attempted; actual bootstrap is tests/bootstrap.php
  • Added inc/class-plugin.php / inc/class-main.php — common WP plugin naming convention that doesn't apply here; main class is inc/class-wp-ultimo.php

Error patterns addressed

Pattern Count Fix
read:file_not_found 38x 3 new not-exist table entries + full test structure documented
bash:other 75x Clearer test structure reduces navigation mistakes
edit:not_read_first 46x Already covered in AGENTS.md; no new gap found
webfetch:other 39x Already covered in AGENTS.md; no new gap found

Testing

Documentation-only change — no PHP/JS code modified.

git ls-files 'tests/Builders/' | head -1         # non-empty confirms directory exists
git ls-files 'inc/bootstrap.php'                  # empty confirms path correctly non-existent
git ls-files 'tests/WP_Ultimo/bootstrap.php'      # empty confirms path correctly non-existent

aidevops.sh v3.13.10 plugin for OpenCode v1.3.17 with claude-sonnet-4-6 spent 6m and 17,830 tokens on this as a headless worker.


Merged via PR #979 to main.
Merged by deterministic merge pass (pulse-wrapper.sh).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

origin:worker Auto-created by pulse labelless backfill (t2112)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Contributor insight: 4 recurring error pattern(s) from superdav42

1 participant