Skip to content

Add base AGENTS.md/CLAUDE.md agent-instructions system - #378

Merged
LayaTaal merged 11 commits into
mainfrom
task/base-agent-instructions
Sep 4, 2026
Merged

Add base AGENTS.md/CLAUDE.md agent-instructions system#378
LayaTaal merged 11 commits into
mainfrom
task/base-agent-instructions

Conversation

@LayaTaal

@LayaTaal LayaTaal commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

What does this do/fix?

Adds a starting point for layered agent instruction system so AI agents (Claude, Codex, Copilot, Cursor) orient themselves faster and allows designers dabbling in code to get a guided, safer workflow.

  • Root instruction files for all 4 platforms (CLAUDE.md, AGENTS.md, .github/copilot-instructions.md, .cursor/rules/project.mdc) — ~60 lines each, covering key paths, stack, persona governance, and hard rules
  • Domain docs loaded on-demand: docs/ai/blocks.md (Gutenberg conventions), docs/ai/php.md (namespaces, DI container, PHPCS), docs/ai/commands.md (lando/npm/wp-cli), docs/ai/designer-mode.md (allowed files, git workflow, regression safety)
  • Persona governance — reads local-config.json; user_role = designer loads designer-mode restrictions silently

Ports the AGENTS.md + docs/ai/ trigger-based instructions set validated
in a client fork back to the base framework, so every future ModernPress
project starts with it. Adapted for moose: PHPCS indentation corrected
to tabs, and the Tests section rewritten to reference Slic instead of
the client fork's lando-composer-test setup. Client-specific hosting
config (Dokku hostnames) was dropped rather than ported.
@LayaTaal
LayaTaal marked this pull request as draft August 13, 2026 14:39
@LayaTaal LayaTaal self-assigned this Aug 24, 2026
@MlKilderkin
MlKilderkin self-requested a review August 24, 2026 15:47

@MlKilderkin MlKilderkin left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Here are some notes
I also added detailed standards for PHP, comments and PRs in #384

Comment thread docs/ai/designer-mode.md Outdated
Comment thread docs/ai/commands.md Outdated
Comment thread docs/ai/commands.md

## Tests

PHP tests run via [Slic](https://github.com/stellarwp/slic) (WP-Browser + Codeception in a Docker container). See `docs/php-tests.md` for full setup.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In most cases that provides you errors and unexpected behaviour because not everyone has slic installed. AI will think that it needs to install

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@MlKilderkin what would you suggest we do here? Is Slic ever really leveraged in ModernPress projects as suggested in php-tests.md? If not, should we just remove this section and do we need to update the PHP testing for ModernPress altogether?

Comment thread docs/ai/php.md
@MlKilderkin

Copy link
Copy Markdown
Contributor

@LayaTaal child PR is merged and we are ready for review

@LayaTaal
LayaTaal marked this pull request as ready for review August 26, 2026 14:54

@dpellenwood dpellenwood left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

This is awesome. I just left a few Q's around hard-oded php versions which can be messy to remember to update later.

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread docs/ai/php.md Outdated
Comment thread docs/ai/php.md Outdated
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@LayaTaal

LayaTaal commented Sep 3, 2026

Copy link
Copy Markdown
Contributor Author

@dpellenwood I updated the docs to check the PHP version instead of hard-coding in the current version.

@jamiepaul jamiepaul left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🙌🏻 great work on this @MlKilderkin & @LayaTaal I love how concise the AGENTS.md is after splitting out various additional context files

@LayaTaal
LayaTaal merged commit ce886dc into main Sep 4, 2026
7 checks passed
@LayaTaal
LayaTaal deleted the task/base-agent-instructions branch September 4, 2026 19:55
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.

5 participants