Skip to content

[ci] Add AGENTS.md, PHP AI skill and remove coverage guards #1767#1768

Merged
michaelvlach merged 4 commits intomainfrom
1767-ci-add-agentsmd
Apr 17, 2026
Merged

[ci] Add AGENTS.md, PHP AI skill and remove coverage guards #1767#1768
michaelvlach merged 4 commits intomainfrom
1767-ci-add-agentsmd

Conversation

@michaelvlach
Copy link
Copy Markdown
Collaborator

Add AGENTS.md documenting repository components, available commands and package names for Rust, TypeScript and PHP. In agdb_api/php/ci.sh, remove leftover .openapi-generator/ directory during the openapi task to clean generated artifacts. Update pnpm-workspace.yaml to remove the agdb_web_next package from the workspace packages list.

Add AGENTS.md documenting repository components, available commands and package names for Rust, TypeScript and PHP. In agdb_api/php/ci.sh, remove leftover .openapi-generator/ directory during the openapi task to clean generated artifacts. Update pnpm-workspace.yaml to remove the agdb_web_next package from the workspace packages list.
Copilot AI review requested due to automatic review settings April 17, 2026 21:43
@michaelvlach michaelvlach linked an issue Apr 17, 2026 that may be closed by this pull request
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
agdb Ready Ready Preview, Comment Apr 17, 2026 10:38pm

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds repository-level agent documentation and performs a couple of small CI/workspace cleanups.

Changes:

  • Add AGENTS.md describing repository components, commands, and package names.
  • Clean PHP OpenAPI generation artifacts by removing the .openapi-generator/ directory during regeneration.
  • Remove agdb_web_next/** from pnpm-workspace.yaml.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
pnpm-workspace.yaml Removes the (now non-present) agdb_web_next package from the pnpm workspace.
agdb_api/php/ci.sh Cleans up .openapi-generator/ during OpenAPI regeneration to avoid leftover artifacts.
AGENTS.md Introduces documentation for components, commands, and package naming across Rust/TypeScript/PHP.

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

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.


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

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Remove strict coverage gating and filename-ignore options from CI llvm-cov invocations, switching the agdb and agdb_server workflow steps to only use --show-missing-lines. Update the coverage workflow invocation to produce lcov output via cargo llvm-cov. Also apply small editorial fixes to AGENTS.md: clarify wording, fix a typo (cargp -> cargo), adjust formatting lines, and add a note about running cargo llvm-cov for package coverage.
@michaelvlach michaelvlach changed the title [ci] Add AGENTS.md #1767 [ci] Add AGENTS.md and remove coverage guards #1767 Apr 17, 2026
Add a new .github/skills/php-composer-setup/SKILL.md that documents installing and verifying PHP and Composer across Windows, Linux, and macOS, outlines verification and repository check commands (for agdb_api/php), and provides usage examples. Update AGENTS.md to include a Setup section with installation hints for Rust, pnpm, PHP/Composer (referencing the new skill), and Playwright, and add TypeScript dependency install/update commands.
@michaelvlach michaelvlach changed the title [ci] Add AGENTS.md and remove coverage guards #1767 [ci] Add AGENTS.md, PHP AI skill and remove coverage guards #1767 Apr 17, 2026
@michaelvlach michaelvlach merged commit 84ccc8b into main Apr 17, 2026
15 checks passed
@michaelvlach michaelvlach deleted the 1767-ci-add-agentsmd branch April 17, 2026 22:42
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.

[ci] Add AGENTS.md

3 participants