Consolidate 35 skills into 10 umbrella skills#60
Merged
pyramation merged 3 commits intomainfrom Mar 18, 2026
Merged
Conversation
Consolidates all standalone skills into 10 umbrella skills to reduce cognitive load and improve agent navigation: 1. constructive - Platform core (safegres, services-schemas, deployment, server-config, functions, cnc-execution-engine, constructive-env) 2. pgpm - stays as-is + absorbs github-workflows-pgpm as CI/CD reference 3. constructive-graphql - codegen, sdk-search, graphql-query, search, pagination 4. constructive-starter-kits - boilerplate-pgpm-init, boilerplate-nextjs-app, boilerplate-authoring 5. constructive-frontend - constructive-ui, crud-stack, meta-forms 6. constructive-testing - pgsql-test, drizzle-orm-test, supabase-test, drizzle-orm, pgsql-parser-testing 7. constructive-ai - pgvector-rag, github-workflows-ollama 8. constructive-tooling - constructive-pnpm, inquirerer-cli, readme-formatting 9. graphile-search - stays as-is (plugin internals, team-level) 10. fbp - fbp-types, fbp-spec, fbp-evaluator, fbp-graph-editor Also deletes planning-blueprinting (not ours). Updates AGENTS.md and CLAUDE.md with new skill structure.
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
…nstructive-graphql)
4 tasks
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
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.
Summary
Reorganizes 35 standalone skill directories into 10 umbrella skills to reduce cognitive load for AI agents navigating this repo. Each umbrella skill has a
SKILL.mdthat acts as a router with a reference table, and absorbed skills become files in the umbrella'sreferences/directory. Also deletesplanning-blueprinting(not ours) and updatesAGENTS.md/CLAUDE.mdwith the new structure.The 10 umbrellas:
constructivepgpmconstructive-graphqlconstructive-starter-kitsconstructive-frontendconstructive-testingconstructive-aiconstructive-toolinggraphile-searchfbpAbsorbed files were moved/renamed only — no content modifications. The new umbrella
SKILL.mdfiles are the only net-new authored content.Updates since last revision:
constructive-aiSDK-first refactorRewrote the
constructive-aiskill to use Constructive SDK + codegen'd ORM patterns only, removing all generic RAG/pgvector tutorials that taught patterns outside the platform (e.g.,new Pool(), rawpgpm docker startsetup).Changes:
SKILL.md— now shows the "Constructive AI Flow" (SDK provision → codegen → ORM query → RAG), with inline code examples usingdb.field.create,db.index.create, anddb.document.findManywithvectorEmbeddingfilterpgvector-rag.md,rag-embeddings.md,rag-similarity-search.md,rag-setup.md— these contained standalonePool-based TypeScript and generic pgvector tutorialsrag-agentic-kit.md→agentic-kit.md,rag-ollama.md→ollama.mdpgvector-sql.md— consolidated raw SQL reference with every code block prefixed-- SQLto clearly distinguish from SDK patternsrag-pipeline.md— end-to-end RAG using SDK provisioning, codegen, ORM ingestion/retrieval, and Ollama generation (includes completeRAGServiceclass)Final reference file set:
rag-pipeline.md,ollama.md,ollama-ci.md,pgvector-sql.md,agentic-kit.mdUpdates since last revision: agentic-kit rewrite + RAG reference for constructive-graphql
agentic-kit.mdfully rewritten to match the actual agentic-kit source. The old version was significantly outdated — it usednew Pool(), referenced a fictionalRAGProviderclass, imported from@sf-ai/agentic-kit, and pointed to deleted skills.New
agentic-kit.mdcovers:AgentKitmanager,OllamaAdapter,AnthropicAdapter,OpenAIAdaptercreateOllamaKit(),createAnthropicKit(),createOpenAIKit(),createMultiProviderKit()agentic-kit,@agentic-kit/ollama,@agentic-kit/anthropic,@agentic-kit/openaiOllamaClient,GenerateInputoptions,AgentProviderinterfaceCreated
constructive-graphql/references/search-rag.md— RAG-specific query patterns using the codegen'd ORM:.update()(not raw SQL)fullTextSearch+vectorEmbeddingin a singlewhereUpdated cross-references:
constructive-graphql/SKILL.md— addedsearch-rag.mdto reference table + cross-references toconstructive-aifor agentic-kit and rag-pipelineconstructive-ai/SKILL.md— addedsearch-rag.mdcross-reference, updated agentic-kit descriptionReview & Testing Checklist for Human
agentic-kit.mdAPI matches actual agentic-kit source: The rewrite usesAgentKit,OllamaAdapter,AnthropicAdapter,OpenAIAdapter, and factory helpers (createOllamaKit, etc.) based on reading the source atpackages/agentic-kit/src/index.ts. Confirm these exports, class names, and method signatures (generate,generateStreaming,addProvider,setProvider) are correct in the published package.search-rag.mdORM patterns: UsesvectorEmbedding: { vector, metric, distance }inwhere,embeddingVectorDistanceas a select field, andmodel.update({ where: { id }, data: { embedding } })for ingestion. These patterns are based on agentic-db'ssearch.tsandembed-all.ts— confirm they match the actual codegen'd ORM output.agentic-kit.md(in constructive-ai) andsearch-rag.md(in constructive-graphql) contain multi-table vector search and 3-pass RAG examples. Verify this duplication is acceptable or whether one should defer to the other more clearly.constructive-ai/SKILL.mdSDK code examples match actual API surface: The Quick Start showsdb.field.create(),db.index.create(), anddb.document.findMany({ where: { vectorEmbedding: { vector, metric, distance } } }). Confirm the ORM actually generatesvectorEmbeddingfilter,EMBEDDING_VECTOR_DISTANCE_ASCorderBy, andembeddingVectorDistanceselect field with these exact names.constructive-ailinks to../constructive-graphql/references/search-rag.md,search-pgvector.md, andsearch-composite.md.constructive-graphqllinks to../constructive-ai/references/agentic-kit.mdandrag-pipeline.md. Confirm all linked files exist on this branch.Suggested test plan: Load the
constructive-aiskill and ask an agent to "build a RAG pipeline with Anthropic" — verify it routes toagentic-kit.mdand producesAgentKit+AnthropicAdaptercode (not the oldRAGProvider/new Pool()patterns). Then loadconstructive-graphqland ask for "vector search for RAG" — verify it routes tosearch-rag.mdwith ORM-based examples.Notes
agentic-kit.md(513 lines) was a complete rewrite (96% changed) — it contained a fictionalRAGProviderclass,@sf-ai/agentic-kitimports,new Pool()database access, and references to 5 deleted skills. None of this matched the actual agentic-kit source.search-rag.mdRAG patterns (VECTOR_CONDITION helper, multi-table parallel search, embedding ingestion via.update()) are modeled on the real implementation in the agentic-db repo (packages/rag/src/search.ts,embed-all.ts).constructive-testinghadframework-details.mdandanti-patterns.mdreference files that were deleted because the rewritten SKILL.md no longer references them. The anti-patterns content was folded into the new SKILL.md itself.constructive-graphql/references/codegen-options.md,codegen.md,search.md,pagination.mdwere created in a prior PR (Consolidate into unifiedconstructive-graphqlskill, delete old codegen + sdk-search #58) and are expected to already exist on the branch.maininto the feature branch to resolve conflicts; also removed a staleconstructive-graphql-codegen.zipthat came in from main (that skill no longer exists).constructive-sdk-agentic-flowrepo'sinstruction.mdwas verified — it already references the new umbrella skill names (constructive-graphqlnotconstructive-graphql-codegen). No changes needed there.Link to Devin session: https://app.devin.ai/sessions/9f711b5939654d0caec800c90bbd073a
Requested by: @pyramation