Skip to content

feat(algolia-audit): add agents/openai.yaml manifest - #42

Open
itsdanwilli wants to merge 1 commit into
mainfrom
audit-openai-manifest
Open

itsdanwilli wants to merge 1 commit into
mainfrom
audit-openai-manifest

Conversation

@itsdanwilli

Copy link
Copy Markdown
Contributor

One file. algolia-audit (#38) is the only skill in the implementation suite without agents/openai.yaml, and the implementation-skills site validator (validate:skills) requires it for every skill it packages — so the site cannot list or ship the audit skill until this lands. Content mirrors the other suite manifests; short_description is within the 25–64 char rule and default_prompt references $algolia-audit.

Unblocks the follow-up PR on algolia/implementation-skills that adds algolia-audit to the site.

🤖 Generated with Claude Code

Every other skill in the implementation suite ships the Codex/ChatGPT interface
manifest; algolia-audit landed without one, which blocks the implementation-skills
site validator from packaging it.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@itsdanwilli
itsdanwilli requested a review from PLNech September 17, 2026 15:28
itsdanwilli added a commit to algolia/implementation-skills that referenced this pull request Sep 17, 2026
….yaml

algolia/skills does not require the Codex interface manifest and its own
validator passes without it; the site should not enforce a stricter shape than
the canonical repo. The manifest is still validated whenever it exists.
algolia-audit currently lacks one (algolia/skills#42 adds it); the weekday sync
will carry it through when that lands.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
itsdanwilli added a commit to algolia/implementation-skills that referenced this pull request Sep 17, 2026
…ocs (#18)

* Add algolia-audit to the site: suite, card, review path, downloads, docs

algolia/skills#38 added `algolia-audit`, the entry point for reviewing an
existing implementation. The site did not know about it: it was not in
packaging/suite.json, so it had no card, no download, no place in the MCP
suite, and the sync workflow would never watch it.

- packaging/suite.json + mcp/skills-pin.json: add algolia-audit to the suite.
- src/main.jsx: new "Audit" card (type QA); the "Review current setup" path
  now starts with algolia-audit before the deep skills.
- README, packaging/library-README.md, artifacts/start-here-prompt.md,
  CLAUDE.md: document the second entry point (builds start with
  discovery-planning, existing implementations start with audit).
- enablement: skill table row (hub) and a customer situation row; pages
  regenerated with the same build scripts.
- public/downloads: algolia-audit.zip added; all archives rebuilt by
  package-downloads (timestamps change, content otherwise identical).

Depends on algolia/skills#42 (agents/openai.yaml for algolia-audit): the
site's validate:skills requires that manifest, so the submodule and MCP pin
must move past that merge before this ships.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* validate:skills — warn, don't fail, when a skill has no agents/openai.yaml

algolia/skills does not require the Codex interface manifest and its own
validator passes without it; the site should not enforce a stricter shape than
the canonical repo. The manifest is still validated whenever it exists.
algolia-audit currently lacks one (algolia/skills#42 adds it); the weekday sync
will carry it through when that lands.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
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.

1 participant