feat(seo): merge claude-seo into the SEO skill catalog + fix skills F…#4
Merged
Merged
Conversation
…TS sync Incorporates the claude-seo repo (MIT, (c) 2026 agricidaniel) into Synapse's git-tracked skill catalog (packages/codegraph/data/skill/), which the container ships and the entrypoint auto-imports on every boot — so this propagates to prod on deploy with no manual step. SEO skills (20): - 11 enriched: seo, seo-technical, seo-audit, seo-geo, seo-schema, seo-content, seo-hreflang, seo-images, seo-sitemap-advanced, seo-competitor-pages, seo-plan (audit/E-E-A-T/GEO methodology, CWV thresholds, falsifiability, primary-source Google guidance; kept our Next.js implementation code) - 6 new: seo-backlinks, seo-local, seo-ecommerce, seo-sxo, seo-content-drift, seo-topical-clusters; +1 reference dataset: seo-google-guidelines - 2 optional generic-named integration refs: seo-rank-data, seo-crawl-data - adapted to our style, no brand names in skill names, MIT attribution footer. FTS fix (the boot-time import was silently corrupting the index every deploy): - skills upsert: INSERT OR REPLACE -> INSERT ... ON CONFLICT(name) DO UPDATE. INSERT OR REPLACE churned skills.rowid on every write, orphaning the FTS5 external-content index (skills_fts) -> `fts5: missing row` in skill_route after each re-import. Stable rowids also preserve autolearning columns for free and keep status on null (importer) vs default active on insert. - migration 035: AFTER INSERT/UPDATE(OF indexed cols)/DELETE triggers maintain FTS via the external-content delete+insert protocol + a one-time rebuild that repairs existing orphans on any DB (incl. prod at deploy). Removed the redundant manual FTS write from code. - +5 regression tests (skills-store-fts). Tests: storage 19, codegraph 294 green. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…TS sync
Incorporates the claude-seo repo (MIT, (c) 2026 agricidaniel) into Synapse's git-tracked skill catalog (packages/codegraph/data/skill/), which the container ships and the entrypoint auto-imports on every boot — so this propagates to prod on deploy with no manual step.
SEO skills (20):
FTS fix (the boot-time import was silently corrupting the index every deploy):
fts5: missing rowin skill_route after each re-import. Stable rowids also preserve autolearning columns for free and keep status on null (importer) vs default active on insert.