Skip to content

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

Merged
itsdanwilli merged 2 commits into
mainfrom
add-algolia-audit
Sep 17, 2026
Merged

itsdanwilli merged 2 commits into
mainfrom
add-algolia-audit

Conversation

@itsdanwilli

@itsdanwilli itsdanwilli commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

algolia-audit landed in algolia/skills (#38) as the entry point for reviewing an existing implementation — the benchmark's headline routing fix (free-choice audits went from invoking 1 skill and matching the no-skills baseline to invoking algolia-audit and fixing 9/10 live-checked defects). The site didn't know it existed: not in packaging/suite.json, so no card, no download, not served by the MCP, and never watched by sync-skills.yml.

What changes

  • packaging/suite.json, mcp/skills-pin.jsonalgolia-audit joins the suite.
  • src/main.jsx — new Audit card (type QA, "START HERE for an existing implementation"); the Review current setup recommended path now opens with algolia-audit, then index-configuration → data-modeling → events-insights → release-qa.
  • README.md, packaging/library-README.md, artifacts/start-here-prompt.md, CLAUDE.md — the suite now has two entry points and the docs say so (builds → discovery-planning; existing implementations → audit).
  • enablement/ — hub skill table row; customer quick-start gets a "You inherited a setup, or it works but nobody has checked it" situation. Regenerated with build.py / build-customer.py exactly as before.
  • public/downloads/algolia-audit.zip added (28 files, 36 KB); every archive is rebuilt by package-downloads so all ZIP timestamps change (same as a sync PR).
  • scripts/validate-skills.mjs — a missing agents/openai.yaml is a warning, not a failure. The canonical repo doesn't require the Codex manifest and its validator passes without it; the site shouldn't enforce a stricter shape. The manifest is still validated when present. algolia-audit is the one skill currently without it — feat(algolia-audit): add agents/openai.yaml manifest skills#42 adds it and the weekday sync will carry it through when it merges; nothing here depends on that.

Validation

validate:skills ✅ (one warning: algolia-audit manifest pending) · validate:packages ✅ (18 archives) · validate:feedback ✅ · vite build

🤖 Generated with Claude Code

itsdanwilli and others added 2 commits September 17, 2026 10:28
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>
….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
itsdanwilli marked this pull request as ready for review September 17, 2026 16:16
@itsdanwilli
itsdanwilli merged commit fae91f5 into main Sep 17, 2026
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