mirror of
https://github.com/CopilotKit/CopilotKit.git
synced 2026-09-14 16:26:20 +08:00
Alem Tuzlak
b691b63660
Substring-match fixtures (pie chart, bar chart, schedule, trip, etc.) cross-fired across demos with different tool surfaces and returned tool names the target agent never registered, causing demos to render nothing in prod when aimock handles traffic. Fixture changes (showcase/aimock/feature-parity.json): - Replace generic pie-chart / bar-chart matches with per-suggestion specific phrases so gen-ui-tool-based gets render_pie_chart / render_bar_chart directly and beautiful-chat gets pieChart / barChart with real data (skipping the query_data two-step that caused the infinite loop on re-matching prompts). - Narrow schedule+meeting to the Beautiful Chat 30-minute prompt returning scheduleTime. - Narrow flight+fly to flights-from-SFO-to-JFK. - Narrow background to sunset-themed-gradient. - Remove trip, sales, pipeline, todo: substring-false-firing across unrelated demos; interrupt/A2UI demos fall through to real LLM. Guardrail (showcase/scripts/validate-fixture-tool-surface.ts): - Pure validate() cross-references every fixture's tool-call name against the tool surface of each demo whose suggestion prompt contains the fixture's match substring. Loud failure when the fixture returns a tool the demo's agent does not register. - CLI walks packages/ collecting suggestions from page.tsx + hooks/, frontend tools from useComponent / useHumanInTheLoop / useFrontendTool / useRenderTool / useDefaultRenderTool, and backend tools via route.ts agentId->graphId map + langgraph.json graph->file + @tool decorators. - 7 vitest cases written TDD-first covering the drift detection, content-only fixtures, case-insensitivity, and multi-tool responses. - Current state: 33 fixtures x 191 demos, no drift. Counterfactual (reverting the pie-chart fix) correctly flags gen-ui-tool-based and declarative-gen-ui. Also fixes a separate runtime bug in the langgraph-python package Dockerfile: WORKDIR /app left /app owned by root; the app user could not create the .langgraph_api cache dir LangGraph's in-memory runtime needs, so the agent crashed on boot. Added a non-recursive chown app:app /app (preserves the original perf intent of the explicit --chown on COPY, which avoided a recursive chown).
79 lines
3.1 KiB
Docker
79 lines
3.1 KiB
Docker
# Stage 1: Build Next.js frontend
|
|
FROM node:22-slim AS frontend
|
|
WORKDIR /app
|
|
COPY package.json ./
|
|
RUN npm install --legacy-peer-deps
|
|
COPY . .
|
|
RUN npm run build
|
|
|
|
# Stage 2: Build Python venv with agent deps
|
|
#
|
|
# True multi-stage split — the builder carries full `python:3.12` (compilers,
|
|
# build-essential, dev headers needed to compile wheels that don't ship
|
|
# pre-built for ``-slim``) and produces a self-contained ``/opt/venv`` that
|
|
# the runner COPYs in as a single opaque tree. The runner never runs
|
|
# ``pip install`` itself, so no compiler toolchain bloats the final image.
|
|
FROM python:3.12.13 AS agent-builder
|
|
WORKDIR /agent
|
|
RUN python -m venv /opt/venv
|
|
ENV PATH=/opt/venv/bin:$PATH
|
|
COPY requirements.txt ./requirements.txt
|
|
RUN pip install --no-cache-dir --upgrade pip \
|
|
&& pip install --no-cache-dir -r requirements.txt
|
|
|
|
# Stage 3: Production image with Node.js + Python (runtime only — no pip,
|
|
# no build tools). Node.js is installed via NodeSource because the package
|
|
# runs BOTH Next.js (frontend) and the Python agent inside this single image;
|
|
# entrypoint.sh orchestrates both processes.
|
|
FROM python:3.12.13-slim AS runner
|
|
RUN apt-get update && apt-get install -y --no-install-recommends \
|
|
curl && \
|
|
curl -fsSL https://deb.nodesource.com/setup_22.x | bash - && \
|
|
apt-get install -y nodejs && \
|
|
apt-get clean && rm -rf /var/lib/apt/lists/*
|
|
|
|
WORKDIR /app
|
|
|
|
# Create unprivileged runtime user BEFORE any COPY so --chown resolves
|
|
# by name and so recursive chown over /app is never needed (fast builds).
|
|
# Mirrors the starter Dockerfile pattern for parity — Railway / any
|
|
# platform that enforces non-root by policy needs this from the package
|
|
# image too, not just the generated starter.
|
|
RUN (groupadd --system --gid 1001 app 2>/dev/null || true) \
|
|
&& (useradd --system --uid 1001 --gid 1001 --no-create-home app 2>/dev/null || true) \
|
|
&& mkdir -p /home/app && chown app:app /home/app \
|
|
&& chown app:app /app
|
|
|
|
# Python venv (prebuilt in agent-builder stage — no pip in the runner).
|
|
COPY --chown=app:app --from=agent-builder /opt/venv /opt/venv
|
|
ENV PATH=/opt/venv/bin:$PATH
|
|
|
|
# Next.js build artifacts
|
|
COPY --chown=app:app --from=frontend /app/.next ./.next
|
|
COPY --chown=app:app --from=frontend /app/node_modules ./node_modules
|
|
COPY --chown=app:app --from=frontend /app/package.json ./
|
|
COPY --chown=app:app --from=frontend /app/public ./public
|
|
|
|
# Agent code + config
|
|
COPY --chown=app:app langgraph.json ./
|
|
COPY --chown=app:app src/agents/ ./src/agents/
|
|
|
|
# Shared Python tools (copied into build context by CI)
|
|
COPY --chown=app:app shared_python/ /app/shared/python/
|
|
ENV PYTHONPATH=/app/shared/python
|
|
|
|
# Entrypoint
|
|
COPY --chown=app:app entrypoint.sh ./
|
|
RUN chmod +x entrypoint.sh
|
|
|
|
USER app
|
|
|
|
EXPOSE 10000
|
|
# Intentionally NOT setting `ENV NODE_ENV=production` at the image level.
|
|
# NODE_ENV=production at the image level would leak into every child process
|
|
# (Python agent, shell scripts, healthchecks) — most of which don't use it
|
|
# the way Next.js does. entrypoint.sh scopes NODE_ENV=production to the
|
|
# Next.js invocation only so non-Next children see the host's environment.
|
|
ENV PORT=10000
|
|
CMD ["./entrypoint.sh"]
|