feat: consolidate dbt skills and add altimate-dbt CLI package#201
feat: consolidate dbt skills and add altimate-dbt CLI package#201anandgupta42 merged 1 commit intomainfrom
Conversation
Claude Code ReviewThis repository is configured for manual code reviews. Comment |
There was a problem hiding this comment.
Pull request overview
This PR introduces a new altimate-dbt CLI package for dbt operations and updates the Altimate builder guidance/tooling to prefer that CLI over the existing raw dbt_run tool, alongside a dependency patch for python-bridge to improve runtime compatibility.
Changes:
- Add
packages/dbt-tools(CLI + adapter) exposingaltimate-dbtcommands likeinit,doctor,compile,build,execute, and DAG helpers. - Remove
dbt_runtool exposure from the opencode tool registry / barrel exports, and update prompts/docs to steer usage towardaltimate-dbt. - Add a
python-bridge@1.1.0patch and wire it viapatchedDependencies.
Reviewed changes
Copilot reviewed 53 out of 55 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| patches/python-bridge@1.1.0.patch | Patch python-bridge to stop promisifying child_process. |
| packages/opencode/src/tool/registry.ts | Removes DbtRunTool from tool registration. |
| packages/opencode/src/altimate/tools/dbt-run.ts | Updates tool description to position it as fallback vs altimate-dbt. |
| packages/opencode/src/altimate/prompts/builder.txt | Rewrites builder prompt to mandate altimate-dbt workflows. |
| packages/opencode/src/altimate/index.ts | Stops exporting dbt-run tool from barrel. |
| packages/dbt-tools/tsconfig.json | New strict TS config for the CLI package. |
| packages/dbt-tools/test/config.test.ts | Adds initial config tests (currently not exercising module behavior). |
| packages/dbt-tools/test/cli.test.ts | Adds CLI smoke tests via bun spawnSync. |
| packages/dbt-tools/src/index.ts | Implements CLI entrypoint/dispatch, output formatting, diagnostics. |
| packages/dbt-tools/src/config.ts | Adds HOME-based config read/write for ~/.altimate-code/dbt.json. |
| packages/dbt-tools/src/commands/init.ts | Adds init command with project + python auto-detection. |
| packages/dbt-tools/src/commands/info.ts | Adds info command wrapper. |
| packages/dbt-tools/src/commands/graph.ts | Adds children/parents command wrappers. |
| packages/dbt-tools/src/commands/execute.ts | Adds execute command with optional --limit. |
| packages/dbt-tools/src/commands/doctor.ts | Adds doctor command (prereq checks summary). |
| packages/dbt-tools/src/commands/deps.ts | Adds deps / add-packages commands. |
| packages/dbt-tools/src/commands/compile.ts | Adds compile / compile-query commands. |
| packages/dbt-tools/src/commands/columns.ts | Adds model/source columns + column-values commands. |
| packages/dbt-tools/src/commands/build.ts | Adds build, run, test, and build-project commands. |
| packages/dbt-tools/src/check.ts | Adds prereq validation logic (python/dbt/project). |
| packages/dbt-tools/src/adapter.ts | Creates DBTProjectIntegrationAdapter wired for local execution. |
| packages/dbt-tools/script/copy-python.ts | Copies bundled Python packages into dist/. |
| packages/dbt-tools/package.json | Defines the altimate-dbt bin + build/test scripts. |
| packages/dbt-tools/bin/altimate-dbt | Node shebang launcher importing built dist entrypoint. |
| package.json | Adds packages/dbt-tools workspace and patched dependency for python-bridge. |
| bun.lock | Locks new workspace package + patched dependency entries. |
| .opencode/skills/yaml-config/SKILL.md | Removes old skill doc. |
| .opencode/skills/model-scaffold/SKILL.md | Removes old skill doc. |
| .opencode/skills/medallion-patterns/SKILL.md | Removes old skill doc. |
| .opencode/skills/incremental-logic/SKILL.md | Removes old skill doc. |
| .opencode/skills/impact-analysis/SKILL.md | Removes old skill doc. |
| .opencode/skills/generate-tests/SKILL.md | Removes old skill doc. |
| .opencode/skills/dbt-troubleshoot/SKILL.md | Adds new troubleshooting skill centered on altimate-dbt. |
| .opencode/skills/dbt-troubleshoot/references/test-failures.md | Adds troubleshooting reference (tests). |
| .opencode/skills/dbt-troubleshoot/references/runtime-errors.md | Adds troubleshooting reference (runtime errors). |
| .opencode/skills/dbt-troubleshoot/references/compilation-errors.md | Adds troubleshooting reference (compile errors). |
| .opencode/skills/dbt-troubleshoot/references/altimate-dbt-commands.md | Adds CLI command reference. |
| .opencode/skills/dbt-test/SKILL.md | Adds new dbt testing skill doc. |
| .opencode/skills/dbt-test/references/unit-test-guide.md | Adds dbt unit test guidance. |
| .opencode/skills/dbt-test/references/schema-test-patterns.md | Adds schema-test patterns reference. |
| .opencode/skills/dbt-test/references/custom-tests.md | Adds custom tests reference. |
| .opencode/skills/dbt-test/references/altimate-dbt-commands.md | Adds CLI command reference (duplicated for skill). |
| .opencode/skills/dbt-docs/SKILL.md | Updates docs skill to use altimate-dbt and new standards. |
| .opencode/skills/dbt-docs/references/documentation-standards.md | Adds documentation standards reference. |
| .opencode/skills/dbt-docs/references/altimate-dbt-commands.md | Adds CLI command reference (for docs skill). |
| .opencode/skills/dbt-develop/SKILL.md | Adds dbt development skill doc. |
| .opencode/skills/dbt-develop/references/yaml-generation.md | Adds YAML generation reference. |
| .opencode/skills/dbt-develop/references/medallion-architecture.md | Adds medallion architecture reference. |
| .opencode/skills/dbt-develop/references/layer-patterns.md | Adds dbt layering reference. |
| .opencode/skills/dbt-develop/references/incremental-strategies.md | Adds incremental strategies reference. |
| .opencode/skills/dbt-develop/references/common-mistakes.md | Adds common mistakes reference. |
| .opencode/skills/dbt-develop/references/altimate-dbt-commands.md | Adds CLI command reference (for develop skill). |
| .opencode/skills/dbt-analyze/SKILL.md | Adds dbt impact analysis skill doc. |
| .opencode/skills/dbt-analyze/references/lineage-interpretation.md | Adds lineage interpretation reference. |
| .opencode/skills/dbt-analyze/references/altimate-dbt-commands.md | Adds CLI command reference (for analyze skill). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Note on
|
c52e6a1 to
2da80e7
Compare
|
This PR doesn't fully meet our contributing guidelines and PR template. What needs to be fixed:
Please edit this PR description to address the above within 2 hours, or it will be automatically closed. If you believe this was flagged incorrectly, please let a maintainer know. |
a7c9919 to
264669a
Compare
✅ Tests — All PassedTypeScript — passedPython — passedTested at |
636b326 to
43b8bd1
Compare
- New `@altimateai/dbt-tools` package with `altimate-dbt` CLI binary - Commands: init, doctor, compile, build, run, test, execute, columns, graph, deps - Auto-detects dbt project, Python environment, and adapter type - Friendly error diagnostics with actionable fix suggestions - Consolidate 6 old skills into 5 reference-backed skills: dbt-develop, dbt-test, dbt-docs, dbt-troubleshoot, dbt-analyze - Update builder prompt to prefer `altimate-dbt` CLI over raw `dbt_run` tool - Remove orphaned `dbt-run.ts` tool file and unused `python-bridge` patch - Fix missing `await` on `children`/`parents` graph commands - Fix `--limit 0` to correctly support schema inspection - Wrap `adapter.dispose()` in try-catch to prevent masking command errors - Skip stub config tests pending rewrite Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
43b8bd1 to
8d1a339
Compare
Discussed with Michiel and this is safe to remove. We have removed it |
What does this PR do?
Consolidates 8 scattered dbt skills into 5 focused, well-structured skills with a
references/system for modular knowledge. Adds a standalonealtimate-dbtCLI package for dbt project operations. Rewrites the builder prompt with a skills-first architecture and leaner token footprint.Type of change
Changes
Skills consolidation (8 → 5):
dbt-analyze,dbt-develop,dbt-docs,dbt-test,dbt-troubleshootreferences/directory with modular knowledge docs (altimate-dbt commands, patterns, error guides)generate-tests,impact-analysis,incremental-logic,medallion-patterns,model-scaffold,yaml-configNew
packages/dbt-toolspackage:altimate-dbtCLI for dbt project operationsbuild,compile,deps,columns,graph,init,doctoraltimate_python_packagesfor Python bridge integrationBuilder prompt rewrite:
altimate_coretoolsOther:
python-bridge@1.1.0patch for packaging fixHow did you verify your code works?
bun turbo typecheck— all 5 packages clean)altimate-dbtCLI boots and resolves commandsChecklist
🤖 Generated with Claude Code