[HYPERSHELL-301] Agentic build chain spec + build-pipeline component - #250
Draft
markturansky wants to merge 6 commits into
Draft
[HYPERSHELL-301] Agentic build chain spec + build-pipeline component#250markturansky wants to merge 6 commits into
markturansky wants to merge 6 commits into
Conversation
Add specs/tooling/build/agentic-build-chain.spec.md defining the full-stack-pipeline workflow as a configurable agentic build chain: typed step graph, per-step model tiers bound by profile, deterministic model-independent gates, run metrics for cross-config comparison, and a new Python/LangChain component under components/. Register it in the spec index.
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository YAML (base), Central YAML (inherited) Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
added 5 commits
September 4, 2026 14:54
…LL-301 Python/LangChain executor for the full-stack-pipeline workflow: - graph.py: pipeline as a typed step DAG mirroring the skill - profiles + models: per-step model bound by profile (tiered/all-small/all-deep), Vertex provider, tool-only steps use no model - gates: deterministic, model-independent acceptance commands - engine: dependency-gated scheduler with retry/escalation, human checkpoints, dry-run, resumable run state - metrics: per-run summary + cross-profile comparison - cli: list/profiles/run/resume/runs/compare/check-drift - tests: 28 deterministic-core unit tests (no LangChain/creds needed) - README: local Vertex run instructions
…RSHELL-301 Add an 'anthropic-vertex' provider (ChatAnthropicVertex from langchain-google-vertexai.model_garden) and repoint the tiered/all-small/ all-deep profiles at Claude models. LangChain does not read ANTHROPIC_VERTEX_PROJECT_ID/CLOUD_ML_REGION, so build_chat_model resolves project/region from those env vars (then GOOGLE_CLOUD_*), passing them explicitly. README updated for the Claude-on-Vertex local run.
ChatAnthropicVertex wraps anthropic.AnthropicVertex, which langchain-google-vertexai does not install. Add anthropic[vertex] so the default Claude-on-Vertex provider constructs without a ModuleNotFoundError.
…PERSHELL-301 Unbounded anthropic[vertex]>=0.39 resolved to 1.3.0, whose messages.create dropped the temperature kwarg that langchain-google-vertexai always sends, breaking every ChatAnthropicVertex call with a TypeError. langchain-google-vertexai requires anthropic<1.0.0; pin to >=0.60,<1. Verified end-to-end: the run now reaches the Vertex API (403 on a dummy project) instead of the TypeError.
…HELL-301 run_llm_step now catches exceptions from llm.invoke (auth, model-not-found, rate limit, timeout) and returns them as the step error, so the engine reports a clean FAIL and applies retry/escalation instead of crashing the run with a traceback.
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.
Summary
Draft for review of the spec (implementation follows on this branch).
Adds
specs/tooling/build/agentic-build-chain.spec.md(HYPERSHELL-301): a behavior contract that recasts the deterministicfull-stack-pipelineskill as a composable agentic build chain. Each workflow step becomes a typed node in a dependency-ordered graph, and each node's model is bound by configuration, not code — mechanical/tool-only steps run cheap/no model, reasoning steps run a deep model. The same graph runs under many profiles; deterministic, model-independent gates hold correctness constant so runs can be compared on cost and latency.What's in this PR
specs/tooling/domain +agentic-build-chain.spec.md(11 requirements AB-01..AB-11, scenarios, design decisions).specs/index.spec.md(Tooling sub-spec + registry row)..gitignorefix: the genericbuild/ignore was swallowingspecs/tooling/build/.Key decisions to review
tool-only(no model) /mechanical→small /reasoning→deep; bound by a profile (tiered/all-small/all-deep), provider-agnostic via LangChain.make/go/generator/e2e commands) so cross-config comparison measures cost/latency at held-constant quality.components/build-pipeline(container-friendly for later sandbox/cluster runs).Follow-up (this branch)
components/build-pipeline: step graph, profiles, gate runner, metrics, run state, CLI. Initial local target: Google Vertex.Spec:
specs/tooling/build/agentic-build-chain.spec.md