Compare commits
28 Commits
v0.3.0
...
7d5289ac54
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
7d5289ac54 | ||
|
|
3d8fc9dacd | ||
|
|
f9f804cd49 | ||
|
|
85f142ade0 | ||
|
|
0dfad02513 | ||
|
|
c44eb680b2 | ||
|
|
38ada998a2 | ||
|
|
74547c2bdf | ||
|
|
587c0d3b1c | ||
|
|
bb61f2992b | ||
|
|
3ba72d9b28 | ||
|
|
b4f0fbc3ea | ||
|
|
12943ee6f4 | ||
|
|
9af95ebd96 | ||
|
|
f0b567f3e6 | ||
|
|
e3d6cf4cf5 | ||
|
|
df59bd010c | ||
|
|
e5152151d6 | ||
|
|
aa2d57e619 | ||
|
|
6b53706987 | ||
|
|
a0cfc866df | ||
|
|
7bf19b6a7b | ||
|
|
19b019a8d8 | ||
|
|
4ef6a22e28 | ||
|
|
3796cfca87 | ||
|
|
7ce544a051 | ||
|
|
391720155e | ||
|
|
ae6600b8d2 |
10
.dockerignore
Normal file
10
.dockerignore
Normal file
@@ -0,0 +1,10 @@
|
|||||||
|
.git
|
||||||
|
.gitea
|
||||||
|
.worktrees
|
||||||
|
.DS_Store
|
||||||
|
*.log
|
||||||
|
.env*
|
||||||
|
.vscode
|
||||||
|
.idea
|
||||||
|
bin/
|
||||||
|
brain/
|
||||||
68
.gitea/workflows/cd.yml
Normal file
68
.gitea/workflows/cd.yml
Normal file
@@ -0,0 +1,68 @@
|
|||||||
|
name: cd
|
||||||
|
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: [main]
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
deploy:
|
||||||
|
name: Build and deploy
|
||||||
|
runs-on: self-hosted
|
||||||
|
env:
|
||||||
|
SERVICE: supervisor
|
||||||
|
IMAGE: gitea.d-ma.be/mathias/supervisor
|
||||||
|
INFRA_REPO: git@gitea.d-ma.be:mathias/infra.git
|
||||||
|
BUILDKIT_HOST: unix:///run/buildkit/buildkitd.sock
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Build and push image
|
||||||
|
run: |
|
||||||
|
set -e
|
||||||
|
trap 'rm -f /tmp/supervisor-image.tar' EXIT
|
||||||
|
IMAGE_TAG="${{ github.sha }}"
|
||||||
|
echo "Building ${IMAGE}:${IMAGE_TAG}"
|
||||||
|
|
||||||
|
# Build to local OCI tar (no registry auth needed at build time)
|
||||||
|
buildctl --addr "${BUILDKIT_HOST}" build \
|
||||||
|
--frontend dockerfile.v0 \
|
||||||
|
--local context=. \
|
||||||
|
--local dockerfile=. \
|
||||||
|
--opt build-arg:VERSION="${IMAGE_TAG}" \
|
||||||
|
--output type=oci,dest=/tmp/supervisor-image.tar
|
||||||
|
|
||||||
|
# Push with skopeo using simple credential flag (avoids OAuth token flow)
|
||||||
|
skopeo copy \
|
||||||
|
oci-archive:/tmp/supervisor-image.tar \
|
||||||
|
docker://${IMAGE}:${IMAGE_TAG} \
|
||||||
|
--dest-creds "${{ secrets.REGISTRY_CREDS }}"
|
||||||
|
|
||||||
|
echo "Built and pushed ${IMAGE}:${IMAGE_TAG}"
|
||||||
|
|
||||||
|
- name: Update infra repo
|
||||||
|
run: |
|
||||||
|
set -e
|
||||||
|
trap 'rm -rf /tmp/infra-update; rm -f ~/.ssh/infra_deploy_key' EXIT
|
||||||
|
IMAGE_TAG="${{ github.sha }}"
|
||||||
|
# Use internal Gitea SSH (runner is on koala — NodePort 30022 is reachable locally)
|
||||||
|
mkdir -p ~/.ssh
|
||||||
|
echo "${{ secrets.INFRA_DEPLOY_KEY }}" > ~/.ssh/infra_deploy_key
|
||||||
|
chmod 600 ~/.ssh/infra_deploy_key
|
||||||
|
printf 'Host gitea.d-ma.be\n HostName 127.0.0.1\n Port 30022\n StrictHostKeyChecking no\n' >> ~/.ssh/config
|
||||||
|
|
||||||
|
GIT_SSH_COMMAND="ssh -i ~/.ssh/infra_deploy_key -o IdentitiesOnly=yes" \
|
||||||
|
git clone "${INFRA_REPO}" /tmp/infra-update
|
||||||
|
|
||||||
|
cd /tmp/infra-update
|
||||||
|
sed -i "s|gitea.d-ma.be/mathias/supervisor:.*|gitea.d-ma.be/mathias/supervisor:${IMAGE_TAG}|" \
|
||||||
|
"k3s/apps/${SERVICE}/deployment.yaml"
|
||||||
|
|
||||||
|
git config user.email "cd-bot@d-ma.be"
|
||||||
|
git config user.name "CD Bot"
|
||||||
|
git add "k3s/apps/${SERVICE}/deployment.yaml"
|
||||||
|
git commit -m "chore(deploy): ${SERVICE} → ${IMAGE_TAG}"
|
||||||
|
GIT_SSH_COMMAND="ssh -i ~/.ssh/infra_deploy_key -o IdentitiesOnly=yes" \
|
||||||
|
git push
|
||||||
|
|
||||||
|
echo "Infra repo updated: ${SERVICE} → ${IMAGE_TAG}"
|
||||||
50
Dockerfile
Normal file
50
Dockerfile
Normal file
@@ -0,0 +1,50 @@
|
|||||||
|
# syntax=docker/dockerfile:1
|
||||||
|
|
||||||
|
# ── Build stage ───────────────────────────────────────────────────────────────
|
||||||
|
FROM golang:1.26-bookworm AS builder
|
||||||
|
|
||||||
|
ARG VERSION=dev
|
||||||
|
WORKDIR /src
|
||||||
|
|
||||||
|
COPY go.mod go.sum ./
|
||||||
|
RUN go mod download
|
||||||
|
|
||||||
|
COPY . .
|
||||||
|
RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 \
|
||||||
|
go build -trimpath -ldflags="-s -w -X main.version=${VERSION}" \
|
||||||
|
-o /out/supervisor ./cmd/supervisor
|
||||||
|
|
||||||
|
# ── Runtime stage ─────────────────────────────────────────────────────────────
|
||||||
|
# Node.js 22 slim — needed for claude CLI subprocess
|
||||||
|
FROM node:22-slim
|
||||||
|
|
||||||
|
# Install claude CLI (provides the `claude` binary the supervisor shells out to)
|
||||||
|
RUN npm install -g @anthropic-ai/claude-code \
|
||||||
|
&& claude --version \
|
||||||
|
&& echo "claude CLI installed"
|
||||||
|
|
||||||
|
# Copy supervisor binary
|
||||||
|
COPY --from=builder /out/supervisor /usr/local/bin/supervisor
|
||||||
|
|
||||||
|
# Bake in config (models.yaml + skill discipline files)
|
||||||
|
COPY config/ /app/config/
|
||||||
|
|
||||||
|
# Run as non-root
|
||||||
|
RUN groupadd -r supervisor && useradd -r -g supervisor -d /app supervisor
|
||||||
|
|
||||||
|
WORKDIR /app
|
||||||
|
|
||||||
|
# brain/ is writable state — mount a PersistentVolume here
|
||||||
|
VOLUME /app/brain
|
||||||
|
|
||||||
|
ENV SUPERVISOR_CONFIG_DIR=/app/config/supervisor
|
||||||
|
ENV SUPERVISOR_MODELS_FILE=/app/config/models.yaml
|
||||||
|
ENV SUPERVISOR_BRAIN_DIR=/app/brain
|
||||||
|
ENV SUPERVISOR_SESSIONS_DIR=/app/brain/sessions
|
||||||
|
ENV SUPERVISOR_PORT=3200
|
||||||
|
|
||||||
|
USER supervisor
|
||||||
|
|
||||||
|
EXPOSE 3200
|
||||||
|
|
||||||
|
ENTRYPOINT ["/usr/local/bin/supervisor"]
|
||||||
@@ -101,7 +101,9 @@ func main() {
|
|||||||
}
|
}
|
||||||
attempts := make([]iexec.AttemptRecord, 0, len(chain))
|
attempts := make([]iexec.AttemptRecord, 0, len(chain))
|
||||||
orch := iexec.NewOrchestrator(chain, litellmExec.Run, claudeExec.Run, verifier, models.LlamaSwapURL(), &attempts)
|
orch := iexec.NewOrchestrator(chain, litellmExec.Run, claudeExec.Run, verifier, models.LlamaSwapURL(), &attempts)
|
||||||
return orch.Run(ctx, req)
|
result, err := orch.Run(ctx, req)
|
||||||
|
result.Attempts = attempts // attach orchestration metadata before returning
|
||||||
|
return result, err
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -164,7 +166,7 @@ func main() {
|
|||||||
mux.Handle("/mcp", srv)
|
mux.Handle("/mcp", srv)
|
||||||
|
|
||||||
addr := ":" + cfg.Port
|
addr := ":" + cfg.Port
|
||||||
logger.Info("supervisor starting", "addr", addr)
|
logger.Info("supervisor starting", "addr", addr, "version", "v0.4.0")
|
||||||
if err := http.ListenAndServe(addr, mux); err != nil {
|
if err := http.ListenAndServe(addr, mux); err != nil {
|
||||||
logger.Error("server stopped", "err", err)
|
logger.Error("server stopped", "err", err)
|
||||||
os.Exit(1)
|
os.Exit(1)
|
||||||
|
|||||||
923
docs/superpowers/plans/2026-04-20-cd-pipeline.md
Normal file
923
docs/superpowers/plans/2026-04-20-cd-pipeline.md
Normal file
@@ -0,0 +1,923 @@
|
|||||||
|
# CD Pipeline Implementation Plan
|
||||||
|
|
||||||
|
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
|
||||||
|
|
||||||
|
**Goal:** Build a GitOps CD pipeline that automatically builds a container image on `main` push and deploys it to k3s on koala via Flux.
|
||||||
|
|
||||||
|
**Architecture:** BuildKit runs as a systemd service on koala (same host as the Gitea runner); CD pushes images to the Gitea registry and commits image tag updates to the infra repo; Flux reconciles within 60s. App secrets (including ANTHROPIC_API_KEY) are SOPS-encrypted in the infra repo and decrypted by Flux at apply time.
|
||||||
|
|
||||||
|
**Tech Stack:** Go 1.26, Node.js 22 (for claude CLI), BuildKit (buildctl), Gitea Actions, Flux (kustomize-controller), SOPS + age, k3s/containerd
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Environment context
|
||||||
|
|
||||||
|
This plan spans three environments. Each task header notes which environment it runs in:
|
||||||
|
|
||||||
|
- **[this-repo]** — `/Users/mathias/Documents/local-dev/AI/supervisor` on flamingo
|
||||||
|
- **[koala-ssh]** — `ssh koala` (run commands via `ssh koala "..."`)
|
||||||
|
- **[infra-repo]** — `gitea.d-ma.be/mathias/infra` (clone to a temp dir, work there, push)
|
||||||
|
- **[gitea-ui]** — Gitea web UI at `https://gitea.d-ma.be`
|
||||||
|
- **[kubectl]** — kubectl from flamingo (home LAN)
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## File map
|
||||||
|
|
||||||
|
**This repo (supervisor):**
|
||||||
|
- Create: `Dockerfile`
|
||||||
|
- Create: `.gitea/workflows/cd.yml`
|
||||||
|
|
||||||
|
**koala host:**
|
||||||
|
- Create: `/etc/systemd/system/buildkitd.service` (or user-level equivalent)
|
||||||
|
- Create: `/root/.config/buildkit/buildkitd.toml` (registry auth config)
|
||||||
|
|
||||||
|
**Infra repo (`gitea.d-ma.be/mathias/infra`):**
|
||||||
|
- Create: `apps/supervisor/namespace.yaml`
|
||||||
|
- Create: `apps/supervisor/deployment.yaml`
|
||||||
|
- Create: `apps/supervisor/service.yaml`
|
||||||
|
- Create: `apps/supervisor/secrets.enc.yaml` (SOPS-encrypted)
|
||||||
|
- Create: `apps/supervisor/kustomization.yaml`
|
||||||
|
- Create: `apps/imagepullsecret/secret.enc.yaml` (SOPS-encrypted)
|
||||||
|
- Create: `apps/imagepullsecret/kustomization.yaml`
|
||||||
|
- Modify: `clusters/koala/kustomization.yaml` (add supervisor + imagepullsecret)
|
||||||
|
- Modify: `flux-system/kustomization.yaml` or relevant Flux Kustomization CRD (add SOPS decryption)
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 1: Dockerfile [this-repo]
|
||||||
|
|
||||||
|
The supervisor binary depends on the `claude` CLI as a subprocess. The image uses a multi-stage build: Go builder stage compiles the binary; the runtime stage is Node.js (for `npm install -g @anthropic-ai/claude-code`). Config files are baked in. The `brain/` directory is a volume mount.
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Create: `Dockerfile`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Verify no Dockerfile exists**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ls Dockerfile 2>/dev/null || echo "confirmed: no Dockerfile"
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: `confirmed: no Dockerfile`
|
||||||
|
|
||||||
|
- [ ] **Step 2: Create the Dockerfile**
|
||||||
|
|
||||||
|
```dockerfile
|
||||||
|
# syntax=docker/dockerfile:1
|
||||||
|
|
||||||
|
# ── Build stage ───────────────────────────────────────────────────────────────
|
||||||
|
FROM golang:1.26-bookworm AS builder
|
||||||
|
|
||||||
|
ARG VERSION=dev
|
||||||
|
WORKDIR /src
|
||||||
|
|
||||||
|
COPY go.mod go.sum ./
|
||||||
|
RUN go mod download
|
||||||
|
|
||||||
|
COPY . .
|
||||||
|
RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 \
|
||||||
|
go build -trimpath -ldflags="-s -w -X main.version=${VERSION}" \
|
||||||
|
-o /out/supervisor ./cmd/supervisor
|
||||||
|
|
||||||
|
# ── Runtime stage ─────────────────────────────────────────────────────────────
|
||||||
|
# Node.js 22 slim — needed for claude CLI subprocess
|
||||||
|
FROM node:22-slim
|
||||||
|
|
||||||
|
# Install claude CLI (provides the `claude` binary the supervisor shells out to)
|
||||||
|
RUN npm install -g @anthropic-ai/claude-code \
|
||||||
|
&& claude --version \
|
||||||
|
&& echo "claude CLI installed"
|
||||||
|
|
||||||
|
# Copy supervisor binary
|
||||||
|
COPY --from=builder /out/supervisor /usr/local/bin/supervisor
|
||||||
|
|
||||||
|
# Bake in config (models.yaml + skill discipline files)
|
||||||
|
COPY config/ /app/config/
|
||||||
|
|
||||||
|
WORKDIR /app
|
||||||
|
|
||||||
|
# brain/ is writable state — mount a PersistentVolume here
|
||||||
|
VOLUME /app/brain
|
||||||
|
|
||||||
|
ENV SUPERVISOR_CONFIG_DIR=/app/config/supervisor
|
||||||
|
ENV SUPERVISOR_MODELS_FILE=/app/config/models.yaml
|
||||||
|
ENV SUPERVISOR_BRAIN_DIR=/app/brain
|
||||||
|
ENV SUPERVISOR_SESSIONS_DIR=/app/brain/sessions
|
||||||
|
ENV SUPERVISOR_PORT=3200
|
||||||
|
|
||||||
|
EXPOSE 3200
|
||||||
|
|
||||||
|
ENTRYPOINT ["/usr/local/bin/supervisor"]
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 3: Build locally to verify it compiles (no push)**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
# buildctl must be available locally, OR use docker if available on flamingo
|
||||||
|
docker build --target builder -t supervisor-build-test . && echo "build stage OK"
|
||||||
|
# If no docker on flamingo, skip this step and verify at Task 3 on koala instead
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add Dockerfile
|
||||||
|
git commit -m "feat: add multi-stage Dockerfile with claude CLI runtime"
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 2: BuildKit systemd service on koala [koala-ssh]
|
||||||
|
|
||||||
|
Install `buildkitd` as a root-level systemd service on koala. The Gitea runner process runs as root (confirmed by PID/cgroup), so the root socket at `/run/buildkit/buildkitd.sock` is accessible to it.
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Create: `/etc/systemd/system/buildkitd.service` on koala
|
||||||
|
- Create: `/etc/buildkit/buildkitd.toml` on koala (registry auth)
|
||||||
|
|
||||||
|
- [ ] **Step 1: Check if buildkitd is already installed**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh koala "buildkitd --version 2>/dev/null || echo 'not installed'"
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Install buildkitd on koala**
|
||||||
|
|
||||||
|
Download the latest buildkit release binary (arm64 or amd64 — koala has x86_64):
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh koala "
|
||||||
|
BUILDKIT_VERSION=v0.21.0
|
||||||
|
curl -sSL https://github.com/moby/buildkit/releases/download/\${BUILDKIT_VERSION}/buildkit-\${BUILDKIT_VERSION}.linux-amd64.tar.gz \
|
||||||
|
| tar -xz -C /usr/local/
|
||||||
|
buildkitd --version
|
||||||
|
"
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected output includes: `buildkitd github.com/moby/buildkit v0.21.0`
|
||||||
|
|
||||||
|
- [ ] **Step 3: Create buildkitd.toml with Gitea registry auth**
|
||||||
|
|
||||||
|
The `[registry]` block configures auth for pushing to `gitea.d-ma.be`. The actual credentials come from `~/.docker/config.json` (which buildkitd reads automatically) — this toml just enables the registry:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh koala "
|
||||||
|
mkdir -p /etc/buildkit
|
||||||
|
cat > /etc/buildkit/buildkitd.toml << 'EOF'
|
||||||
|
[worker.containerd]
|
||||||
|
enabled = false
|
||||||
|
|
||||||
|
[worker.oci]
|
||||||
|
enabled = true
|
||||||
|
|
||||||
|
[registry.\"gitea.d-ma.be\"]
|
||||||
|
http = false
|
||||||
|
insecure = false
|
||||||
|
EOF
|
||||||
|
"
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Create systemd unit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh koala "
|
||||||
|
cat > /etc/systemd/system/buildkitd.service << 'EOF'
|
||||||
|
[Unit]
|
||||||
|
Description=BuildKit daemon
|
||||||
|
After=network.target
|
||||||
|
|
||||||
|
[Service]
|
||||||
|
Type=notify
|
||||||
|
ExecStart=/usr/local/bin/buildkitd --config /etc/buildkit/buildkitd.toml
|
||||||
|
Restart=on-failure
|
||||||
|
LimitNOFILE=1048576
|
||||||
|
LimitNPROC=1048576
|
||||||
|
|
||||||
|
[Install]
|
||||||
|
WantedBy=multi-user.target
|
||||||
|
EOF
|
||||||
|
systemctl daemon-reload
|
||||||
|
systemctl enable buildkitd
|
||||||
|
systemctl start buildkitd
|
||||||
|
"
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 5: Verify the socket exists and is responsive**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh koala "
|
||||||
|
systemctl status buildkitd --no-pager
|
||||||
|
buildctl --addr unix:///run/buildkit/buildkitd.sock debug info
|
||||||
|
"
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: service `active (running)`, buildctl shows BuildKit version info.
|
||||||
|
|
||||||
|
- [ ] **Step 6: Smoke-test build with trivial Dockerfile**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh koala "
|
||||||
|
echo 'FROM alpine:3.21
|
||||||
|
RUN echo hello' | buildctl --addr unix:///run/buildkit/buildkitd.sock build \
|
||||||
|
--frontend dockerfile.v0 \
|
||||||
|
--local context=/ \
|
||||||
|
--opt filename=Dockerfile \
|
||||||
|
--output type=image,name=localhost/smoke-test:latest
|
||||||
|
echo 'smoke test OK'
|
||||||
|
"
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: `smoke test OK`
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 3: Gitea registry push auth for buildctl [koala-ssh]
|
||||||
|
|
||||||
|
`buildctl` reads Docker-style credentials from `/root/.docker/config.json`. Create the credentials file so the runner can push to `gitea.d-ma.be`.
|
||||||
|
|
||||||
|
**Prerequisites:** A Gitea user token or password with `write:packages` scope for the `mathias` org. Create one in Gitea → User Settings → Applications → Generate Token (scopes: `write:packages`).
|
||||||
|
|
||||||
|
- [ ] **Step 1: Create Gitea access token**
|
||||||
|
|
||||||
|
In Gitea UI (`https://gitea.d-ma.be`) → top-right avatar → Settings → Applications → Generate New Token.
|
||||||
|
- Token name: `buildkit-push`
|
||||||
|
- Scopes: `write:packages` (container registry write)
|
||||||
|
- Copy the token — it won't be shown again.
|
||||||
|
|
||||||
|
- [ ] **Step 2: Write docker config.json on koala**
|
||||||
|
|
||||||
|
Replace `<TOKEN>` with the token from Step 1:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh koala "
|
||||||
|
mkdir -p /root/.docker
|
||||||
|
TOKEN=<TOKEN>
|
||||||
|
AUTH=\$(echo -n 'mathias:'\${TOKEN} | base64)
|
||||||
|
cat > /root/.docker/config.json << EOF
|
||||||
|
{
|
||||||
|
\"auths\": {
|
||||||
|
\"gitea.d-ma.be\": {
|
||||||
|
\"auth\": \"\${AUTH}\"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
EOF
|
||||||
|
chmod 600 /root/.docker/config.json
|
||||||
|
echo 'credentials written'
|
||||||
|
"
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 3: Verify push works**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh koala "
|
||||||
|
echo 'FROM alpine:3.21' | buildctl --addr unix:///run/buildkit/buildkitd.sock build \
|
||||||
|
--frontend dockerfile.v0 \
|
||||||
|
--local context=/ \
|
||||||
|
--opt filename=Dockerfile \
|
||||||
|
--output type=image,name=gitea.d-ma.be/mathias/supervisor:push-test,push=true
|
||||||
|
echo 'push OK'
|
||||||
|
"
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: `push OK`. Verify in Gitea UI: `https://gitea.d-ma.be/mathias/supervisor/packages` should show a `push-test` tag.
|
||||||
|
|
||||||
|
- [ ] **Step 4: Delete the test image tag**
|
||||||
|
|
||||||
|
In Gitea UI → supervisor repo → Packages tab → delete the `push-test` tag.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 4: age keypair + Flux SOPS decryption [kubectl + flamingo]
|
||||||
|
|
||||||
|
Flux decrypts SOPS-encrypted secrets at apply time. It needs the age private key stored as a k8s Secret in the `flux-system` namespace.
|
||||||
|
|
||||||
|
- [ ] **Step 1: Verify age is installed**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
age --version || brew install age
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Generate age keypair**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
age-keygen -o /tmp/supervisor-age.key
|
||||||
|
cat /tmp/supervisor-age.key
|
||||||
|
```
|
||||||
|
|
||||||
|
Output includes two lines:
|
||||||
|
```
|
||||||
|
# public key: age1xxxxxx...
|
||||||
|
AGE-SECRET-KEY-1xxxxxxx...
|
||||||
|
```
|
||||||
|
|
||||||
|
**Copy the public key** (the `age1...` value) — you'll need it in Task 7 for encrypting secrets.
|
||||||
|
**Store the private key file securely** — back it up outside the cluster (e.g., 1Password or encrypted note).
|
||||||
|
|
||||||
|
- [ ] **Step 3: Create the SOPS age secret in flux-system**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
kubectl create secret generic sops-age \
|
||||||
|
--from-file=age.agekey=/tmp/supervisor-age.key \
|
||||||
|
-n flux-system
|
||||||
|
kubectl get secret sops-age -n flux-system
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: secret exists with `age.agekey` key.
|
||||||
|
|
||||||
|
- [ ] **Step 4: Shred the temp key file**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
shred -u /tmp/supervisor-age.key
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 5: Check what Flux Kustomization CRDs exist in the infra repo**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git clone git@gitea.d-ma.be:mathias/infra.git /tmp/infra-sops-setup
|
||||||
|
ls /tmp/infra-sops-setup/flux-system/
|
||||||
|
```
|
||||||
|
|
||||||
|
Look for a `kustomization.yaml` or `gotk-sync.yaml` that defines the main Flux Kustomization resource pointing at the `clusters/koala/` path.
|
||||||
|
|
||||||
|
- [ ] **Step 6: Patch the Flux Kustomization to enable SOPS decryption**
|
||||||
|
|
||||||
|
Find the Kustomization resource that syncs `clusters/koala/`. It will look like:
|
||||||
|
|
||||||
|
```yaml
|
||||||
|
apiVersion: kustomize.toolkit.fluxcd.io/v1
|
||||||
|
kind: Kustomization
|
||||||
|
metadata:
|
||||||
|
name: flux-system
|
||||||
|
namespace: flux-system
|
||||||
|
spec:
|
||||||
|
path: ./clusters/koala
|
||||||
|
...
|
||||||
|
```
|
||||||
|
|
||||||
|
Add the `decryption` block:
|
||||||
|
|
||||||
|
```yaml
|
||||||
|
decryption:
|
||||||
|
provider: sops
|
||||||
|
secretRef:
|
||||||
|
name: sops-age
|
||||||
|
```
|
||||||
|
|
||||||
|
Edit the file in `/tmp/infra-sops-setup/flux-system/` and commit:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cd /tmp/infra-sops-setup
|
||||||
|
# Edit the relevant Kustomization yaml to add decryption block (shown above)
|
||||||
|
git add flux-system/
|
||||||
|
git commit -m "feat: enable SOPS decryption via age key in flux-system"
|
||||||
|
git push
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 7: Verify Flux picks up the change**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
flux reconcile source git flux-system
|
||||||
|
flux get kustomizations
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: `flux-system` Kustomization shows `Ready True` with no errors.
|
||||||
|
|
||||||
|
- [ ] **Step 8: Clean up temp clone**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
rm -rf /tmp/infra-sops-setup
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 5: Infra repo — supervisor app manifests [infra-repo]
|
||||||
|
|
||||||
|
Create the full k8s manifest set for the supervisor service in the infra repo. The deployment uses an `IMAGE_TAG` placeholder; the CD job patches this with the actual git sha before pushing.
|
||||||
|
|
||||||
|
**Prerequisites:** age public key from Task 4 Step 2.
|
||||||
|
|
||||||
|
- [ ] **Step 1: Clone the infra repo**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git clone git@gitea.d-ma.be:mathias/infra.git /tmp/infra-supervisor
|
||||||
|
cd /tmp/infra-supervisor
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Create namespace**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
mkdir -p apps/supervisor
|
||||||
|
cat > apps/supervisor/namespace.yaml << 'EOF'
|
||||||
|
apiVersion: v1
|
||||||
|
kind: Namespace
|
||||||
|
metadata:
|
||||||
|
name: supervisor
|
||||||
|
EOF
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 3: Create deployment**
|
||||||
|
|
||||||
|
The `brain` volume is a `hostPath` on koala (simplest for a single-node service; add a PVC later if needed). The image uses `imagePullSecrets` to pull from the Gitea registry.
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cat > apps/supervisor/deployment.yaml << 'EOF'
|
||||||
|
apiVersion: apps/v1
|
||||||
|
kind: Deployment
|
||||||
|
metadata:
|
||||||
|
name: supervisor
|
||||||
|
namespace: supervisor
|
||||||
|
spec:
|
||||||
|
replicas: 1
|
||||||
|
selector:
|
||||||
|
matchLabels:
|
||||||
|
app: supervisor
|
||||||
|
template:
|
||||||
|
metadata:
|
||||||
|
labels:
|
||||||
|
app: supervisor
|
||||||
|
spec:
|
||||||
|
nodeSelector:
|
||||||
|
kubernetes.io/hostname: koala
|
||||||
|
imagePullSecrets:
|
||||||
|
- name: gitea-registry
|
||||||
|
containers:
|
||||||
|
- name: supervisor
|
||||||
|
image: gitea.d-ma.be/mathias/supervisor:IMAGE_TAG
|
||||||
|
ports:
|
||||||
|
- containerPort: 3200
|
||||||
|
envFrom:
|
||||||
|
- secretRef:
|
||||||
|
name: supervisor-secrets
|
||||||
|
env:
|
||||||
|
- name: SUPERVISOR_PORT
|
||||||
|
value: "3200"
|
||||||
|
- name: LITELLM_BASE_URL
|
||||||
|
value: "http://iguana:4000"
|
||||||
|
- name: LLAMA_SWAP_URL
|
||||||
|
value: "http://koala:8080"
|
||||||
|
- name: INGEST_BASE_URL
|
||||||
|
value: "http://localhost:3300"
|
||||||
|
volumeMounts:
|
||||||
|
- name: brain
|
||||||
|
mountPath: /app/brain
|
||||||
|
volumes:
|
||||||
|
- name: brain
|
||||||
|
hostPath:
|
||||||
|
path: /var/lib/supervisor/brain
|
||||||
|
type: DirectoryOrCreate
|
||||||
|
EOF
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Create service**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cat > apps/supervisor/service.yaml << 'EOF'
|
||||||
|
apiVersion: v1
|
||||||
|
kind: Service
|
||||||
|
metadata:
|
||||||
|
name: supervisor
|
||||||
|
namespace: supervisor
|
||||||
|
spec:
|
||||||
|
selector:
|
||||||
|
app: supervisor
|
||||||
|
ports:
|
||||||
|
- port: 3200
|
||||||
|
targetPort: 3200
|
||||||
|
type: ClusterIP
|
||||||
|
EOF
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 5: Create kustomization.yaml for supervisor**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cat > apps/supervisor/kustomization.yaml << 'EOF'
|
||||||
|
apiVersion: kustomize.config.k8s.io/v1beta1
|
||||||
|
kind: Kustomization
|
||||||
|
resources:
|
||||||
|
- namespace.yaml
|
||||||
|
- deployment.yaml
|
||||||
|
- service.yaml
|
||||||
|
- secrets.enc.yaml
|
||||||
|
EOF
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 6: Ensure clusters/koala/kustomization.yaml exists and includes supervisor**
|
||||||
|
|
||||||
|
Check if the file exists:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cat clusters/koala/kustomization.yaml 2>/dev/null || echo "need to create"
|
||||||
|
```
|
||||||
|
|
||||||
|
If it exists, add supervisor and imagepullsecret resources. If it does not exist, create it:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cat > clusters/koala/kustomization.yaml << 'EOF'
|
||||||
|
apiVersion: kustomize.config.k8s.io/v1beta1
|
||||||
|
kind: Kustomization
|
||||||
|
resources:
|
||||||
|
- ../../apps/imagepullsecret
|
||||||
|
- ../../apps/supervisor
|
||||||
|
EOF
|
||||||
|
```
|
||||||
|
|
||||||
|
If it already exists, add the two resource lines (preserving existing entries).
|
||||||
|
|
||||||
|
- [ ] **Step 7: Commit (without secrets — those come in Task 6)**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cd /tmp/infra-supervisor
|
||||||
|
git add apps/supervisor/ clusters/koala/
|
||||||
|
git commit -m "feat(supervisor): add k8s manifests for supervisor service"
|
||||||
|
git push
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 6: SOPS-encrypted secrets in infra repo [infra-repo + flamingo]
|
||||||
|
|
||||||
|
Two encrypted secret files: the imagePullSecret for the Gitea container registry, and the supervisor app secrets (ANTHROPIC_API_KEY, LITELLM_API_KEY).
|
||||||
|
|
||||||
|
**Prerequisites:**
|
||||||
|
- age public key from Task 4 Step 2 (format: `age1xxxxx...`)
|
||||||
|
- `sops` installed (`brew install sops` if missing)
|
||||||
|
- Gitea registry token (same one used in Task 3, or create a read-only one for pulling)
|
||||||
|
|
||||||
|
- [ ] **Step 1: Verify sops is installed**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
sops --version || brew install sops
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Create .sops.yaml in infra repo root**
|
||||||
|
|
||||||
|
This tells sops which key to use for all files in the repo:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cd /tmp/infra-supervisor
|
||||||
|
cat > .sops.yaml << 'EOF'
|
||||||
|
creation_rules:
|
||||||
|
- age: age1REPLACE_WITH_YOUR_PUBLIC_KEY
|
||||||
|
EOF
|
||||||
|
git add .sops.yaml
|
||||||
|
git commit -m "chore: add sops config (age key)"
|
||||||
|
git push
|
||||||
|
```
|
||||||
|
|
||||||
|
Replace `age1REPLACE_WITH_YOUR_PUBLIC_KEY` with the actual age public key from Task 4.
|
||||||
|
|
||||||
|
- [ ] **Step 3: Create and encrypt the imagePullSecret**
|
||||||
|
|
||||||
|
The imagePullSecret is a namespace-less Secret (it will be targeted per namespace via Kustomize). Create it in the `imagepullsecret` app:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
mkdir -p apps/imagepullsecret
|
||||||
|
|
||||||
|
# Create a registry pull token in Gitea: Settings → Applications → Generate Token
|
||||||
|
# Scopes: read:packages
|
||||||
|
# Use that token here (or reuse the buildkit-push token — read access is enough for pulling)
|
||||||
|
PULL_TOKEN=<gitea-read-packages-token>
|
||||||
|
PULL_AUTH=$(echo -n "mathias:${PULL_TOKEN}" | base64)
|
||||||
|
|
||||||
|
cat > /tmp/gitea-pull-secret.yaml << EOF
|
||||||
|
apiVersion: v1
|
||||||
|
kind: Secret
|
||||||
|
metadata:
|
||||||
|
name: gitea-registry
|
||||||
|
namespace: supervisor
|
||||||
|
type: kubernetes.io/dockerconfigjson
|
||||||
|
stringData:
|
||||||
|
.dockerconfigjson: |
|
||||||
|
{
|
||||||
|
"auths": {
|
||||||
|
"gitea.d-ma.be": {
|
||||||
|
"auth": "${PULL_AUTH}"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
EOF
|
||||||
|
|
||||||
|
sops --encrypt /tmp/gitea-pull-secret.yaml > apps/imagepullsecret/secret.enc.yaml
|
||||||
|
rm /tmp/gitea-pull-secret.yaml
|
||||||
|
|
||||||
|
cat > apps/imagepullsecret/kustomization.yaml << 'EOF'
|
||||||
|
apiVersion: kustomize.config.k8s.io/v1beta1
|
||||||
|
kind: Kustomization
|
||||||
|
resources:
|
||||||
|
- secret.enc.yaml
|
||||||
|
EOF
|
||||||
|
```
|
||||||
|
|
||||||
|
Verify the encrypted file looks correct (should show `sops:` metadata at the bottom):
|
||||||
|
|
||||||
|
```bash
|
||||||
|
tail -20 apps/imagepullsecret/secret.enc.yaml
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Create and encrypt supervisor app secrets**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
# ANTHROPIC_API_KEY: your Anthropic API key
|
||||||
|
# LITELLM_API_KEY: the key your LiteLLM instance expects (can be any string if it's local)
|
||||||
|
cat > /tmp/supervisor-secrets.yaml << 'EOF'
|
||||||
|
apiVersion: v1
|
||||||
|
kind: Secret
|
||||||
|
metadata:
|
||||||
|
name: supervisor-secrets
|
||||||
|
namespace: supervisor
|
||||||
|
type: Opaque
|
||||||
|
stringData:
|
||||||
|
ANTHROPIC_API_KEY: "REPLACE_WITH_REAL_KEY"
|
||||||
|
LITELLM_API_KEY: "REPLACE_WITH_REAL_KEY"
|
||||||
|
EOF
|
||||||
|
|
||||||
|
# Edit /tmp/supervisor-secrets.yaml to insert real values, then:
|
||||||
|
sops --encrypt /tmp/supervisor-secrets.yaml > apps/supervisor/secrets.enc.yaml
|
||||||
|
rm /tmp/supervisor-secrets.yaml
|
||||||
|
```
|
||||||
|
|
||||||
|
Verify:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
tail -20 apps/supervisor/secrets.enc.yaml
|
||||||
|
# Should show encrypted values and sops metadata
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 5: Commit encrypted secrets**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cd /tmp/infra-supervisor
|
||||||
|
git add apps/imagepullsecret/ apps/supervisor/secrets.enc.yaml .sops.yaml
|
||||||
|
git commit -m "feat: add SOPS-encrypted imagePullSecret and supervisor app secrets"
|
||||||
|
git push
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 6: Verify Flux reconciles and creates the secrets**
|
||||||
|
|
||||||
|
Wait ~60s then:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
flux reconcile kustomization flux-system --with-source
|
||||||
|
kubectl get secrets -n supervisor
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: `gitea-registry` and `supervisor-secrets` appear in the `supervisor` namespace.
|
||||||
|
|
||||||
|
- [ ] **Step 7: Clean up temp clone**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
rm -rf /tmp/infra-supervisor
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 7: Gitea org-level secrets [gitea-ui + koala-ssh]
|
||||||
|
|
||||||
|
Set the three secrets that all repos in the `mathias` org will inherit. These go in the Gitea org (not individual repos).
|
||||||
|
|
||||||
|
**Files:** No files — Gitea UI configuration.
|
||||||
|
|
||||||
|
- [ ] **Step 1: Generate SSH deploy key for infra repo**
|
||||||
|
|
||||||
|
On flamingo:
|
||||||
|
|
||||||
|
```bash
|
||||||
|
ssh-keygen -t ed25519 -C "cd-bot infra deploy key" -f /tmp/infra-deploy-key -N ""
|
||||||
|
cat /tmp/infra-deploy-key # private key → INFRA_DEPLOY_KEY secret
|
||||||
|
cat /tmp/infra-deploy-key.pub # public key → add to Gitea infra repo as deploy key
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Add public key to infra repo as a deploy key (write access)**
|
||||||
|
|
||||||
|
In Gitea UI: `https://gitea.d-ma.be/mathias/infra` → Settings → Deploy Keys → Add Deploy Key.
|
||||||
|
- Title: `cd-bot`
|
||||||
|
- Key: paste content of `/tmp/infra-deploy-key.pub`
|
||||||
|
- Enable write access: ✓
|
||||||
|
|
||||||
|
- [ ] **Step 3: Set org-level secrets in Gitea**
|
||||||
|
|
||||||
|
In Gitea UI: `https://gitea.d-ma.be/org/mathias/settings/secrets` → Add Secret.
|
||||||
|
|
||||||
|
Set these three secrets:
|
||||||
|
|
||||||
|
| Secret name | Value |
|
||||||
|
|-------------|-------|
|
||||||
|
| `INFRA_DEPLOY_KEY` | content of `/tmp/infra-deploy-key` (private key, including `-----BEGIN...` lines) |
|
||||||
|
| `BUILDKIT_REGISTRY_AUTH` | same base64 auth string as used in Task 3 Step 2 (format: `mathias:<token>` base64-encoded) |
|
||||||
|
|
||||||
|
Note: `BUILDKIT_REGISTRY_AUTH` is redundant if `/root/.docker/config.json` is already on the runner host from Task 3 — but setting it as a secret allows the `cd.yml` to explicitly pass it to `buildctl` for clarity and rotation.
|
||||||
|
|
||||||
|
- [ ] **Step 4: Clean up temp key files**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
shred -u /tmp/infra-deploy-key /tmp/infra-deploy-key.pub
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 5: Verify secrets appear in Gitea**
|
||||||
|
|
||||||
|
In Gitea UI: `https://gitea.d-ma.be/org/mathias/settings/secrets` — confirm both secrets are listed (values are hidden, only names shown).
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 8: cd.yml workflow [this-repo]
|
||||||
|
|
||||||
|
Create the CD workflow that triggers after CI passes, builds the image with buildctl, and commits the updated tag to the infra repo.
|
||||||
|
|
||||||
|
**Files:**
|
||||||
|
- Create: `.gitea/workflows/cd.yml`
|
||||||
|
|
||||||
|
- [ ] **Step 1: Create cd.yml**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
cat > .gitea/workflows/cd.yml << 'EOF'
|
||||||
|
name: cd
|
||||||
|
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: [main]
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
deploy:
|
||||||
|
name: Build and deploy
|
||||||
|
needs: [check] # 'check' is the job name in ci.yml
|
||||||
|
runs-on: self-hosted
|
||||||
|
env:
|
||||||
|
SERVICE: supervisor
|
||||||
|
REGISTRY: gitea.d-ma.be
|
||||||
|
IMAGE: gitea.d-ma.be/mathias/supervisor
|
||||||
|
INFRA_REPO: git@gitea.d-ma.be:mathias/infra.git
|
||||||
|
BUILDKIT_HOST: unix:///run/buildkit/buildkitd.sock
|
||||||
|
steps:
|
||||||
|
- name: Checkout
|
||||||
|
uses: actions/checkout@v4
|
||||||
|
|
||||||
|
- name: Build and push image
|
||||||
|
run: |
|
||||||
|
IMAGE_TAG="${{ github.sha }}"
|
||||||
|
echo "Building ${IMAGE}:${IMAGE_TAG}"
|
||||||
|
buildctl --addr "${BUILDKIT_HOST}" build \
|
||||||
|
--frontend dockerfile.v0 \
|
||||||
|
--local context=. \
|
||||||
|
--local dockerfile=. \
|
||||||
|
--opt build-arg:VERSION="${IMAGE_TAG}" \
|
||||||
|
--output "type=image,name=${IMAGE}:${IMAGE_TAG},push=true"
|
||||||
|
echo "IMAGE_TAG=${IMAGE_TAG}" >> $GITHUB_OUTPUT
|
||||||
|
id: build
|
||||||
|
|
||||||
|
- name: Update infra repo
|
||||||
|
run: |
|
||||||
|
IMAGE_TAG="${{ github.sha }}"
|
||||||
|
# Write SSH key for infra repo
|
||||||
|
mkdir -p ~/.ssh
|
||||||
|
echo "${{ secrets.INFRA_DEPLOY_KEY }}" > ~/.ssh/infra_deploy_key
|
||||||
|
chmod 600 ~/.ssh/infra_deploy_key
|
||||||
|
ssh-keyscan gitea.d-ma.be >> ~/.ssh/known_hosts 2>/dev/null
|
||||||
|
|
||||||
|
# Clone infra repo
|
||||||
|
GIT_SSH_COMMAND="ssh -i ~/.ssh/infra_deploy_key -o IdentitiesOnly=yes" \
|
||||||
|
git clone "${INFRA_REPO}" /tmp/infra-update
|
||||||
|
|
||||||
|
# Patch the image tag
|
||||||
|
cd /tmp/infra-update
|
||||||
|
sed -i "s|gitea.d-ma.be/mathias/supervisor:.*|gitea.d-ma.be/mathias/supervisor:${IMAGE_TAG}|" \
|
||||||
|
"apps/${SERVICE}/deployment.yaml"
|
||||||
|
|
||||||
|
# Commit and push
|
||||||
|
git config user.email "cd-bot@d-ma.be"
|
||||||
|
git config user.name "CD Bot"
|
||||||
|
git add "apps/${SERVICE}/deployment.yaml"
|
||||||
|
git commit -m "chore(deploy): ${SERVICE} → ${IMAGE_TAG}"
|
||||||
|
GIT_SSH_COMMAND="ssh -i ~/.ssh/infra_deploy_key -o IdentitiesOnly=yes" \
|
||||||
|
git push
|
||||||
|
|
||||||
|
# Clean up
|
||||||
|
rm -rf /tmp/infra-update
|
||||||
|
rm ~/.ssh/infra_deploy_key
|
||||||
|
echo "Infra repo updated: ${SERVICE} → ${IMAGE_TAG}"
|
||||||
|
EOF
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Verify the `needs` job name matches ci.yml**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
grep "^ [a-z].*:$" .gitea/workflows/ci.yml
|
||||||
|
```
|
||||||
|
|
||||||
|
The output should show `check:` as the quality-gate job name. The `cd.yml` uses `needs: [check]` — confirm this matches.
|
||||||
|
|
||||||
|
- [ ] **Step 3: Commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git add .gitea/workflows/cd.yml
|
||||||
|
git commit -m "feat: add CD workflow (buildctl → Gitea registry → infra repo update)"
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 9: End-to-end smoke test
|
||||||
|
|
||||||
|
Trigger the full pipeline and verify each stage.
|
||||||
|
|
||||||
|
- [ ] **Step 1: Push to main to trigger CI + CD**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git push origin main
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 2: Monitor CI job in Gitea**
|
||||||
|
|
||||||
|
Open `https://gitea.d-ma.be/mathias/supervisor/actions` — wait for the `ci` workflow `check` job to pass.
|
||||||
|
|
||||||
|
- [ ] **Step 3: Monitor CD job**
|
||||||
|
|
||||||
|
In the same actions view, the `cd` workflow should start after `ci` passes. Check the `Build and push image` step output for:
|
||||||
|
|
||||||
|
```
|
||||||
|
Building gitea.d-ma.be/mathias/supervisor:<sha>
|
||||||
|
```
|
||||||
|
|
||||||
|
And the `Update infra repo` step for:
|
||||||
|
|
||||||
|
```
|
||||||
|
Infra repo updated: supervisor → <sha>
|
||||||
|
```
|
||||||
|
|
||||||
|
- [ ] **Step 4: Verify image in Gitea registry**
|
||||||
|
|
||||||
|
```
|
||||||
|
https://gitea.d-ma.be/mathias/supervisor/packages
|
||||||
|
```
|
||||||
|
|
||||||
|
Should show a new tag matching the commit sha.
|
||||||
|
|
||||||
|
- [ ] **Step 5: Verify infra repo commit**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
git clone git@gitea.d-ma.be:mathias/infra.git /tmp/infra-verify
|
||||||
|
cd /tmp/infra-verify
|
||||||
|
git log --oneline -3
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: most recent commit message is `chore(deploy): supervisor → <sha>`.
|
||||||
|
|
||||||
|
```bash
|
||||||
|
grep "image:" apps/supervisor/deployment.yaml
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: `image: gitea.d-ma.be/mathias/supervisor:<sha>`
|
||||||
|
|
||||||
|
- [ ] **Step 6: Verify Flux reconciles**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
flux get kustomizations
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: `flux-system` shows `Ready True` and `Applied revision: main/<infra-sha>`.
|
||||||
|
|
||||||
|
```bash
|
||||||
|
kubectl get pods -n supervisor
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: supervisor pod is `Running` with the new image sha.
|
||||||
|
|
||||||
|
- [ ] **Step 7: Verify pod started correctly**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
kubectl logs -n supervisor deployment/supervisor --tail=20
|
||||||
|
```
|
||||||
|
|
||||||
|
Expected: supervisor startup logs (MCP server listening on port 3200, no errors).
|
||||||
|
|
||||||
|
- [ ] **Step 8: Clean up verify clone**
|
||||||
|
|
||||||
|
```bash
|
||||||
|
rm -rf /tmp/infra-verify
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Task 10: Post-deploy — registry retention policy [gitea-ui]
|
||||||
|
|
||||||
|
Prevent the Gitea container registry from filling up by setting a tag retention policy.
|
||||||
|
|
||||||
|
- [ ] **Step 1: Set tag retention in Gitea**
|
||||||
|
|
||||||
|
In Gitea UI: `https://gitea.d-ma.be/mathias/supervisor` → Settings → Packages → Container Registry.
|
||||||
|
|
||||||
|
Set: Keep last **20** tags per image name.
|
||||||
|
|
||||||
|
If Gitea does not expose a UI retention policy, note this for manual cleanup and open a task to automate it (e.g., a weekly Actions job that calls `docker image prune` via the Gitea API).
|
||||||
|
|
||||||
|
- [ ] **Step 2: Verify existing test tags are cleaned up**
|
||||||
|
|
||||||
|
Manually delete any test tags pushed during Task 3 if not already done.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Self-review checklist (for plan author — not a task)
|
||||||
|
|
||||||
|
- [x] **Spec coverage:** BuildKit systemd ✓, cd.yml ✓, Flux SOPS ✓, infra repo structure ✓, imagePullSecret ✓, app secrets ✓, Gitea org secrets ✓, error handling (implicit in workflow failures) ✓, registry retention ✓, smoke test ✓
|
||||||
|
- [x] **Placeholders:** `REPLACE_WITH_YOUR_PUBLIC_KEY` and `REPLACE_WITH_REAL_KEY` are intentional — real values come from user's secrets; marked clearly
|
||||||
|
- [x] **Type consistency:** No shared types across tasks (infra-only plan)
|
||||||
|
- [x] **Known gaps:** `needs: [check]` assumes ci.yml job name is `check` — verified in Task 8 Step 2. The `sed` image tag patch assumes no other image line in deployment.yaml — the deployment template only has one `image:` line.
|
||||||
218
docs/superpowers/specs/2026-04-20-cd-pipeline-design.md
Normal file
218
docs/superpowers/specs/2026-04-20-cd-pipeline-design.md
Normal file
@@ -0,0 +1,218 @@
|
|||||||
|
# CD Pipeline Design
|
||||||
|
|
||||||
|
**Date:** 2026-04-20
|
||||||
|
**Status:** Approved for implementation
|
||||||
|
|
||||||
|
## Problem statement
|
||||||
|
|
||||||
|
The supervisor (and future services on the koala k3s cluster) have no automated deployment path after CI passes. Images are not built, the cluster is updated manually, and there is no audit trail for what is running where.
|
||||||
|
|
||||||
|
## Goal
|
||||||
|
|
||||||
|
After a push to `main` passes CI, automatically build a container image, push it to the Gitea registry, and update the cluster via GitOps — with a design that scales to many repos and services without per-repo kubeconfig or secret sprawl.
|
||||||
|
|
||||||
|
## Success criteria
|
||||||
|
|
||||||
|
- [ ] Successful `main` push triggers image build and push to `gitea.d-ma.be/<org>/<repo>:<git-sha>`
|
||||||
|
- [ ] Infra repo receives a commit updating the image tag for the deployed service
|
||||||
|
- [ ] Flux reconciles within 60s of the infra repo commit; pod runs the new image
|
||||||
|
- [ ] Rollback = one commit to infra repo reverting the tag
|
||||||
|
- [ ] Secrets (app secrets, registry pull) are SOPS-encrypted in infra repo; no manual `kubectl create secret`
|
||||||
|
- [ ] Adding a new service requires only: adding `apps/<service>/` to infra repo + `cd.yml` to the app repo
|
||||||
|
- [ ] Zero changes to the k3s cluster networking or runner configuration
|
||||||
|
|
||||||
|
## Constraints
|
||||||
|
|
||||||
|
- Gitea Actions self-hosted runner runs as a **systemd host process** on koala — not a k8s pod; cannot use cluster DNS
|
||||||
|
- k3s uses containerd; no Docker daemon, no nerdctl on koala
|
||||||
|
- Flux is already running (core controllers only); image-reflector/image-automation are NOT installed and will NOT be added
|
||||||
|
- SOPS + age is the secret management standard; no plaintext Secrets in git
|
||||||
|
- All org-level Gitea secrets are shared across repos — minimize the set
|
||||||
|
|
||||||
|
## Out of scope
|
||||||
|
|
||||||
|
- Multi-cluster promotion (koala only for now; infra repo structure supports adding clusters later)
|
||||||
|
- Automated rollback on health check failure (manual rollback via infra repo commit)
|
||||||
|
- Build caching beyond BuildKit's local disk cache
|
||||||
|
- PR preview environments
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Architecture
|
||||||
|
|
||||||
|
```
|
||||||
|
App repo (supervisor, n8n, etc.)
|
||||||
|
↓ push to main
|
||||||
|
Gitea Actions — ci.yml (lint + test)
|
||||||
|
↓ passes
|
||||||
|
Gitea Actions — cd.yml
|
||||||
|
├─ 1. buildctl → BuildKit (unix socket on koala host)
|
||||||
|
│ → pushes gitea.d-ma.be/<org>/<repo>:<git-sha>
|
||||||
|
├─ 2. Clone infra repo (SSH deploy key)
|
||||||
|
│ → patch apps/<service>/deployment.yaml IMAGE_TAG → <git-sha>
|
||||||
|
│ → git commit + push
|
||||||
|
└─ done
|
||||||
|
|
||||||
|
gitea.d-ma.be/mathias/infra (Flux source)
|
||||||
|
↓ Flux source-controller detects new commit (30s interval)
|
||||||
|
kustomize-controller
|
||||||
|
└─ applies apps/<service>/kustomization.yaml → k3s namespace
|
||||||
|
↓
|
||||||
|
pod runs new image (pulls from gitea.d-ma.be with imagePullSecret)
|
||||||
|
```
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Components
|
||||||
|
|
||||||
|
### 1. BuildKit — systemd service on koala
|
||||||
|
|
||||||
|
BuildKit runs as a rootless systemd service on the koala host, identical to the Gitea runner pattern already in use.
|
||||||
|
|
||||||
|
- Socket: `unix:///run/user/<uid>/buildkit/buildkitd.sock` (rootless) or `/run/buildkit/buildkitd.sock` (root)
|
||||||
|
- Cache: local disk at default BuildKit cache path — persists across builds
|
||||||
|
- Access: `buildctl --addr unix:///run/buildkit/buildkitd.sock` from the runner process (same host, same user)
|
||||||
|
- No k3s involvement for builds
|
||||||
|
|
||||||
|
### 2. Gitea Actions — `cd.yml`
|
||||||
|
|
||||||
|
Separate workflow file; triggers on `main` push after `ci.yml` succeeds.
|
||||||
|
|
||||||
|
```yaml
|
||||||
|
name: cd
|
||||||
|
on:
|
||||||
|
push:
|
||||||
|
branches: [main]
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
deploy:
|
||||||
|
needs: [ci] # or workflow_run trigger — see implementation plan
|
||||||
|
runs-on: [self-hosted, koala]
|
||||||
|
env:
|
||||||
|
IMAGE: gitea.d-ma.be/${{ github.repository }}:${{ github.sha }}
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
- name: Build and push
|
||||||
|
run: |
|
||||||
|
buildctl --addr unix:///run/buildkit/buildkitd.sock \
|
||||||
|
build \
|
||||||
|
--frontend dockerfile.v0 \
|
||||||
|
--local context=. \
|
||||||
|
--local dockerfile=. \
|
||||||
|
--output type=image,name=$IMAGE,push=true
|
||||||
|
env:
|
||||||
|
BUILDKIT_HOST: unix:///run/buildkit/buildkitd.sock
|
||||||
|
- name: Update infra repo
|
||||||
|
run: |
|
||||||
|
git clone git@gitea.d-ma.be:mathias/infra.git /tmp/infra
|
||||||
|
cd /tmp/infra
|
||||||
|
sed -i "s|IMAGE_TAG|${{ github.sha }}|g" apps/${{ env.SERVICE_NAME }}/deployment.yaml
|
||||||
|
git config user.email "cd-bot@d-ma.be"
|
||||||
|
git config user.name "CD Bot"
|
||||||
|
git add apps/${{ env.SERVICE_NAME }}/deployment.yaml
|
||||||
|
git commit -m "chore(deploy): ${{ env.SERVICE_NAME }} → ${{ github.sha }}"
|
||||||
|
git push
|
||||||
|
env:
|
||||||
|
GIT_SSH_COMMAND: ssh -i /tmp/infra-deploy-key -o StrictHostKeyChecking=no
|
||||||
|
```
|
||||||
|
|
||||||
|
`SERVICE_NAME` is set per-repo (either hardcoded in `cd.yml` or derived from the repo name).
|
||||||
|
|
||||||
|
### 3. Org-level Gitea secrets
|
||||||
|
|
||||||
|
Three secrets, set once, inherited by all repos:
|
||||||
|
|
||||||
|
| Secret | Purpose |
|
||||||
|
|--------|---------|
|
||||||
|
| `BUILDKIT_REGISTRY_AUTH` | credentials for pushing to `gitea.d-ma.be` (buildctl `--opt` or `~/.docker/config.json`) |
|
||||||
|
| `INFRA_DEPLOY_KEY` | SSH private key with write access to `gitea.d-ma.be/mathias/infra` |
|
||||||
|
| `KUBECONFIG_KOALA` | (optional) kubeconfig for manual `kubectl` steps if ever needed; scoped ServiceAccount |
|
||||||
|
|
||||||
|
### 4. Infra repo structure
|
||||||
|
|
||||||
|
```
|
||||||
|
gitea.d-ma.be/mathias/infra
|
||||||
|
├── clusters/
|
||||||
|
│ └── koala/
|
||||||
|
│ └── kustomization.yaml # points at ../../apps/*/
|
||||||
|
├── apps/
|
||||||
|
│ ├── supervisor/
|
||||||
|
│ │ ├── namespace.yaml
|
||||||
|
│ │ ├── deployment.yaml # image: gitea.d-ma.be/mathias/supervisor:IMAGE_TAG
|
||||||
|
│ │ ├── service.yaml
|
||||||
|
│ │ ├── secrets.enc.yaml # SOPS-encrypted app secrets (ANTHROPIC_API_KEY, etc.)
|
||||||
|
│ │ └── kustomization.yaml
|
||||||
|
│ ├── n8n/
|
||||||
|
│ │ └── ...
|
||||||
|
│ └── imagepullsecret/
|
||||||
|
│ └── secret.enc.yaml # SOPS-encrypted imagePullSecret for gitea.d-ma.be
|
||||||
|
└── flux-system/ # existing Flux bootstrap manifests
|
||||||
|
```
|
||||||
|
|
||||||
|
Adding a new service = add `apps/<service>/` directory. The `clusters/koala/kustomization.yaml` uses a glob or explicit list.
|
||||||
|
|
||||||
|
### 5. SOPS + age for Flux
|
||||||
|
|
||||||
|
Flux decrypts SOPS-encrypted files at apply time using an age key stored as a k8s Secret in the `flux-system` namespace. Setup:
|
||||||
|
|
||||||
|
1. Generate age keypair: `age-keygen`
|
||||||
|
2. Store private key: `kubectl create secret generic sops-age --from-file=age.agekey -n flux-system`
|
||||||
|
3. Configure Flux Kustomization with `decryption.provider: sops`
|
||||||
|
4. Encrypt secrets before committing: `sops --encrypt --age <pubkey> secret.yaml > secret.enc.yaml`
|
||||||
|
|
||||||
|
App secrets (e.g., `ANTHROPIC_API_KEY`) and the registry pull secret live as encrypted files in `apps/<service>/` and `apps/imagepullsecret/` respectively.
|
||||||
|
|
||||||
|
### 6. Image pull secret
|
||||||
|
|
||||||
|
Each app namespace needs a `kubernetes.io/dockerconfigjson` Secret to pull from `gitea.d-ma.be`. This Secret is SOPS-encrypted in `apps/imagepullsecret/` and applied to each app namespace via Kustomize `namespace` field or a shared Kustomize component.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Data flow: supervisor deploy
|
||||||
|
|
||||||
|
1. Push to `supervisor` main → CI passes (lint/test/vet)
|
||||||
|
2. CD job builds image: `gitea.d-ma.be/mathias/supervisor:abc1234`
|
||||||
|
3. CD job clones infra repo, patches `apps/supervisor/deployment.yaml`, commits
|
||||||
|
4. Flux source-controller detects infra commit within 30s
|
||||||
|
5. kustomize-controller applies `apps/supervisor/kustomization.yaml`
|
||||||
|
6. Flux decrypts `secrets.enc.yaml` → k8s Secret in `supervisor` namespace
|
||||||
|
7. k3s pulls `gitea.d-ma.be/mathias/supervisor:abc1234` using imagePullSecret
|
||||||
|
8. Pod starts with new image; previous pod terminates
|
||||||
|
|
||||||
|
Rollback: `git revert <tag-commit>` in infra repo → Flux reconciles → old image deployed.
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Error handling
|
||||||
|
|
||||||
|
| Scenario | Behaviour |
|
||||||
|
|----------|-----------|
|
||||||
|
| CI fails | `cd.yml` does not run (`needs: ci` gate) |
|
||||||
|
| BuildKit unreachable | `buildctl` exits non-zero → workflow fails; infra repo untouched |
|
||||||
|
| Image push fails | Workflow fails; infra repo untouched; cluster unchanged |
|
||||||
|
| Infra repo push conflict | Retry once with rebase; fail and alert if still conflicting |
|
||||||
|
| Flux reconcile error | Notification-controller fires alert; pods stay on previous image |
|
||||||
|
| Pod image pull fails | `ImagePullBackOff`; Flux reports degraded Kustomization |
|
||||||
|
| SOPS decrypt fails | Kustomization fails; Flux reports error; no partial apply |
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Testing approach
|
||||||
|
|
||||||
|
1. **BuildKit smoke test** — `buildctl build` with a trivial one-line Dockerfile; verify image appears in Gitea registry
|
||||||
|
2. **cd.yml dry run** — trigger manually on a test branch; verify infra repo commit contains correct sha
|
||||||
|
3. **Flux reconcile test** — push infra commit; verify `flux get kustomizations` shows `Ready` and pod runs new image sha
|
||||||
|
4. **Pull secret test** — delete pod, verify it restarts and pulls from Gitea registry without `ImagePullBackOff`
|
||||||
|
5. **SOPS round-trip test** — encrypt a dummy secret, push to infra repo, verify Flux decrypts and `kubectl get secret` shows correct data
|
||||||
|
|
||||||
|
---
|
||||||
|
|
||||||
|
## Risks
|
||||||
|
|
||||||
|
| Risk | Mitigation |
|
||||||
|
|------|------------|
|
||||||
|
| BuildKit socket path varies by user/rootless mode | Confirm path during setup; hardcode in `cd.yml` |
|
||||||
|
| Infra repo concurrent pushes (multiple repos deploying simultaneously) | Git rebase retry handles this; unlikely at current scale |
|
||||||
|
| age private key lost | Back up to SOPS-accessible location; document recovery procedure |
|
||||||
|
| Registry storage fills up | Set Gitea registry tag retention policy (keep last 20 per repo) |
|
||||||
|
| Gitea deploy key compromised | Rotate via Gitea UI; single key for infra repo only |
|
||||||
@@ -17,6 +17,7 @@ type Result struct {
|
|||||||
Verified bool `json:"verified"` // based on exit code, never self-report
|
Verified bool `json:"verified"` // based on exit code, never self-report
|
||||||
ModelUsed string `json:"model_used"` // model name or "self"
|
ModelUsed string `json:"model_used"` // model name or "self"
|
||||||
Message string `json:"message"` // one sentence summary
|
Message string `json:"message"` // one sentence summary
|
||||||
|
Attempts []AttemptRecord `json:"attempts,omitempty"` // populated by orchestrator, not Claude
|
||||||
}
|
}
|
||||||
|
|
||||||
var validStatuses = map[string]bool{"pass": true, "fail": true, "error": true}
|
var validStatuses = map[string]bool{"pass": true, "fail": true, "error": true}
|
||||||
|
|||||||
26
internal/session/attempts.go
Normal file
26
internal/session/attempts.go
Normal file
@@ -0,0 +1,26 @@
|
|||||||
|
// internal/session/attempts.go
|
||||||
|
package session
|
||||||
|
|
||||||
|
import iexec "github.com/mathiasbq/supervisor/internal/exec"
|
||||||
|
|
||||||
|
// AttemptsFrom converts exec.AttemptRecord slice to session.Attempt slice
|
||||||
|
// for writing into a session JSONL entry.
|
||||||
|
func AttemptsFrom(records []iexec.AttemptRecord) []Attempt {
|
||||||
|
if len(records) == 0 {
|
||||||
|
return nil
|
||||||
|
}
|
||||||
|
out := make([]Attempt, len(records))
|
||||||
|
for i, r := range records {
|
||||||
|
out[i] = Attempt{
|
||||||
|
Attempt: i + 1,
|
||||||
|
Model: r.Model,
|
||||||
|
Tier: r.Tier,
|
||||||
|
DurationMs: r.DurationMs,
|
||||||
|
WarmStart: r.WarmStart,
|
||||||
|
Verdict: r.Verdict,
|
||||||
|
Feedback: r.Feedback,
|
||||||
|
Verified: r.Verdict == "accept",
|
||||||
|
}
|
||||||
|
}
|
||||||
|
return out
|
||||||
|
}
|
||||||
37
internal/session/attempts_test.go
Normal file
37
internal/session/attempts_test.go
Normal file
@@ -0,0 +1,37 @@
|
|||||||
|
package session_test
|
||||||
|
|
||||||
|
import (
|
||||||
|
"testing"
|
||||||
|
|
||||||
|
"github.com/mathiasbq/supervisor/internal/exec"
|
||||||
|
"github.com/mathiasbq/supervisor/internal/session"
|
||||||
|
"github.com/stretchr/testify/assert"
|
||||||
|
"github.com/stretchr/testify/require"
|
||||||
|
)
|
||||||
|
|
||||||
|
func TestAttemptsFromEmpty(t *testing.T) {
|
||||||
|
result := session.AttemptsFrom(nil)
|
||||||
|
assert.Empty(t, result)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestAttemptsFromSetsIndex(t *testing.T) {
|
||||||
|
records := []exec.AttemptRecord{
|
||||||
|
{Model: "ollama/phi4", Tier: "local", DurationMs: 1200, WarmStart: true, Verdict: "escalate", Feedback: "too vague"},
|
||||||
|
{Model: "claude-sonnet-4-6", Tier: "subagent", DurationMs: 3400, WarmStart: false, Verdict: "accept"},
|
||||||
|
}
|
||||||
|
result := session.AttemptsFrom(records)
|
||||||
|
require.Len(t, result, 2)
|
||||||
|
|
||||||
|
assert.Equal(t, 1, result[0].Attempt)
|
||||||
|
assert.Equal(t, "ollama/phi4", result[0].Model)
|
||||||
|
assert.Equal(t, "local", result[0].Tier)
|
||||||
|
assert.Equal(t, int64(1200), result[0].DurationMs)
|
||||||
|
assert.True(t, result[0].WarmStart)
|
||||||
|
assert.Equal(t, "escalate", result[0].Verdict)
|
||||||
|
assert.Equal(t, "too vague", result[0].Feedback)
|
||||||
|
assert.False(t, result[0].Verified)
|
||||||
|
|
||||||
|
assert.Equal(t, 2, result[1].Attempt)
|
||||||
|
assert.Equal(t, "claude-sonnet-4-6", result[1].Model)
|
||||||
|
assert.True(t, result[1].Verified)
|
||||||
|
}
|
||||||
@@ -36,3 +36,21 @@ func FormatHistory(entries []Entry, excludePhase string) string {
|
|||||||
}
|
}
|
||||||
return b.String()
|
return b.String()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// PrependHistory reads the session log for sessionID and prepends a formatted
|
||||||
|
// history block to task. Returns task unchanged if sessionID or sessionsDir is
|
||||||
|
// empty, or if no prior entries exist.
|
||||||
|
func PrependHistory(sessionsDir, sessionID, currentPhase, task string) string {
|
||||||
|
if sessionID == "" || sessionsDir == "" {
|
||||||
|
return task
|
||||||
|
}
|
||||||
|
entries, err := Read(sessionsDir, sessionID)
|
||||||
|
if err != nil || len(entries) == 0 {
|
||||||
|
return task
|
||||||
|
}
|
||||||
|
history := FormatHistory(entries, currentPhase)
|
||||||
|
if history == "" {
|
||||||
|
return task
|
||||||
|
}
|
||||||
|
return history + "\n---\n\n" + task
|
||||||
|
}
|
||||||
|
|||||||
@@ -2,11 +2,13 @@
|
|||||||
package session_test
|
package session_test
|
||||||
|
|
||||||
import (
|
import (
|
||||||
|
"strings"
|
||||||
"testing"
|
"testing"
|
||||||
"time"
|
"time"
|
||||||
|
|
||||||
"github.com/mathiasbq/supervisor/internal/session"
|
"github.com/mathiasbq/supervisor/internal/session"
|
||||||
"github.com/stretchr/testify/assert"
|
"github.com/stretchr/testify/assert"
|
||||||
|
"github.com/stretchr/testify/require"
|
||||||
)
|
)
|
||||||
|
|
||||||
func TestFormatHistoryEmpty(t *testing.T) {
|
func TestFormatHistoryEmpty(t *testing.T) {
|
||||||
@@ -39,3 +41,45 @@ func TestFormatHistoryExcludesCurrentPhase(t *testing.T) {
|
|||||||
assert.Contains(t, result, "red done")
|
assert.Contains(t, result, "red done")
|
||||||
assert.NotContains(t, result, "green done")
|
assert.NotContains(t, result, "green done")
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func TestPrependHistoryNoSessionID(t *testing.T) {
|
||||||
|
result := session.PrependHistory("", "", "review", "do the task")
|
||||||
|
assert.Equal(t, "do the task", result)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestPrependHistoryNoLog(t *testing.T) {
|
||||||
|
dir := t.TempDir()
|
||||||
|
result := session.PrependHistory(dir, "sess-abc", "review", "do the task")
|
||||||
|
assert.Equal(t, "do the task", result)
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestPrependHistoryPrependsHistory(t *testing.T) {
|
||||||
|
dir := t.TempDir()
|
||||||
|
entry := session.Entry{
|
||||||
|
SessionID: "sess-abc", Skill: "tdd", Phase: "red",
|
||||||
|
FinalStatus: "pass", Message: "wrote test",
|
||||||
|
Timestamp: time.Now(),
|
||||||
|
}
|
||||||
|
require.NoError(t, session.Append(dir, "sess-abc", entry))
|
||||||
|
|
||||||
|
result := session.PrependHistory(dir, "sess-abc", "review", "do the task")
|
||||||
|
assert.Contains(t, result, "## Session history")
|
||||||
|
assert.Contains(t, result, "wrote test")
|
||||||
|
assert.True(t, strings.HasSuffix(result, "do the task"))
|
||||||
|
}
|
||||||
|
|
||||||
|
func TestPrependHistoryExcludesCurrentPhase(t *testing.T) {
|
||||||
|
dir := t.TempDir()
|
||||||
|
require.NoError(t, session.Append(dir, "sess-abc", session.Entry{
|
||||||
|
SessionID: "sess-abc", Skill: "tdd", Phase: "red",
|
||||||
|
FinalStatus: "pass", Message: "red done", Timestamp: time.Now(),
|
||||||
|
}))
|
||||||
|
require.NoError(t, session.Append(dir, "sess-abc", session.Entry{
|
||||||
|
SessionID: "sess-abc", Skill: "tdd", Phase: "green",
|
||||||
|
FinalStatus: "pass", Message: "green done", Timestamp: time.Now(),
|
||||||
|
}))
|
||||||
|
|
||||||
|
result := session.PrependHistory(dir, "sess-abc", "green", "do the task")
|
||||||
|
assert.Contains(t, result, "red done")
|
||||||
|
assert.NotContains(t, result, "green done")
|
||||||
|
}
|
||||||
|
|||||||
@@ -5,6 +5,7 @@ import (
|
|||||||
"context"
|
"context"
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
"fmt"
|
"fmt"
|
||||||
|
"time"
|
||||||
|
|
||||||
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
||||||
"github.com/mathiasbq/supervisor/internal/session"
|
"github.com/mathiasbq/supervisor/internal/session"
|
||||||
@@ -43,11 +44,12 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
"phase: debug\nproject_root: %s\nerror: %s\ncontext: %s\nmodel: %s",
|
"phase: debug\nproject_root: %s\nerror: %s\ncontext: %s\nmodel: %s",
|
||||||
a.ProjectRoot, a.Error, a.Context, model,
|
a.ProjectRoot, a.Error, a.Context, model,
|
||||||
)
|
)
|
||||||
task = s.prependHistory(a.SessionID, "debug", task)
|
task = session.PrependHistory(s.cfg.SessionsDir, a.SessionID, "debug", task)
|
||||||
|
|
||||||
if s.cfg.ExecutorFn == nil {
|
if s.cfg.ExecutorFn == nil {
|
||||||
return nil, fmt.Errorf("no executor configured")
|
return nil, fmt.Errorf("no executor configured")
|
||||||
}
|
}
|
||||||
|
t0 := time.Now()
|
||||||
result, err := s.cfg.ExecutorFn(ctx, iexec.Request{
|
result, err := s.cfg.ExecutorFn(ctx, iexec.Request{
|
||||||
SkillPrompt: s.cfg.SkillPrompt,
|
SkillPrompt: s.cfg.SkillPrompt,
|
||||||
TaskPrompt: task,
|
TaskPrompt: task,
|
||||||
@@ -57,24 +59,25 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if a.SessionID != "" && s.cfg.SessionsDir != "" {
|
||||||
|
_ = session.Append(s.cfg.SessionsDir, a.SessionID, session.Entry{
|
||||||
|
SessionID: a.SessionID,
|
||||||
|
Timestamp: time.Now(),
|
||||||
|
Skill: "debug",
|
||||||
|
Phase: "debug",
|
||||||
|
ProjectRoot: a.ProjectRoot,
|
||||||
|
Attempts: session.AttemptsFrom(result.Attempts),
|
||||||
|
FinalStatus: result.Status,
|
||||||
|
ModelUsed: result.ModelUsed,
|
||||||
|
DurationMs: time.Since(t0).Milliseconds(),
|
||||||
|
Message: result.Message,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
b, err := json.Marshal(result)
|
b, err := json.Marshal(result)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, fmt.Errorf("marshal result: %w", err)
|
return nil, fmt.Errorf("marshal result: %w", err)
|
||||||
}
|
}
|
||||||
return b, nil
|
return b, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
func (s *Skill) prependHistory(sessionID, currentPhase, task string) string {
|
|
||||||
if sessionID == "" || s.cfg.SessionsDir == "" {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
entries, err := session.Read(s.cfg.SessionsDir, sessionID)
|
|
||||||
if err != nil || len(entries) == 0 {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
history := session.FormatHistory(entries, currentPhase)
|
|
||||||
if history == "" {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
return history + "\n---\n\n" + task
|
|
||||||
}
|
|
||||||
|
|||||||
@@ -5,6 +5,7 @@ import (
|
|||||||
"context"
|
"context"
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
"fmt"
|
"fmt"
|
||||||
|
"time"
|
||||||
|
|
||||||
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
||||||
"github.com/mathiasbq/supervisor/internal/session"
|
"github.com/mathiasbq/supervisor/internal/session"
|
||||||
@@ -52,6 +53,7 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
if s.cfg.ExecutorFn == nil {
|
if s.cfg.ExecutorFn == nil {
|
||||||
return nil, fmt.Errorf("no executor configured")
|
return nil, fmt.Errorf("no executor configured")
|
||||||
}
|
}
|
||||||
|
t0 := time.Now()
|
||||||
result, err := s.cfg.ExecutorFn(ctx, iexec.Request{
|
result, err := s.cfg.ExecutorFn(ctx, iexec.Request{
|
||||||
SkillPrompt: s.cfg.SkillPrompt,
|
SkillPrompt: s.cfg.SkillPrompt,
|
||||||
TaskPrompt: taskPrompt,
|
TaskPrompt: taskPrompt,
|
||||||
@@ -62,6 +64,18 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
return nil, fmt.Errorf("retrospective worker: %w", err)
|
return nil, fmt.Errorf("retrospective worker: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
_ = session.Append(s.cfg.SessionsDir, a.SessionID, session.Entry{
|
||||||
|
SessionID: a.SessionID,
|
||||||
|
Timestamp: time.Now(),
|
||||||
|
Skill: "retrospective",
|
||||||
|
Phase: "retrospective",
|
||||||
|
Attempts: session.AttemptsFrom(result.Attempts),
|
||||||
|
FinalStatus: result.Status,
|
||||||
|
ModelUsed: result.ModelUsed,
|
||||||
|
DurationMs: time.Since(t0).Milliseconds(),
|
||||||
|
Message: result.Message,
|
||||||
|
})
|
||||||
|
|
||||||
b, err := json.Marshal(result)
|
b, err := json.Marshal(result)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, fmt.Errorf("marshal result: %w", err)
|
return nil, fmt.Errorf("marshal result: %w", err)
|
||||||
|
|||||||
@@ -6,6 +6,7 @@ import (
|
|||||||
"encoding/json"
|
"encoding/json"
|
||||||
"fmt"
|
"fmt"
|
||||||
"strings"
|
"strings"
|
||||||
|
"time"
|
||||||
|
|
||||||
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
||||||
"github.com/mathiasbq/supervisor/internal/session"
|
"github.com/mathiasbq/supervisor/internal/session"
|
||||||
@@ -44,11 +45,12 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
"phase: review\nproject_root: %s\nfiles: %s\ncontext: %s\nmodel: %s",
|
"phase: review\nproject_root: %s\nfiles: %s\ncontext: %s\nmodel: %s",
|
||||||
a.ProjectRoot, strings.Join(a.Files, ", "), a.Context, model,
|
a.ProjectRoot, strings.Join(a.Files, ", "), a.Context, model,
|
||||||
)
|
)
|
||||||
task = s.prependHistory(a.SessionID, "review", task)
|
task = session.PrependHistory(s.cfg.SessionsDir, a.SessionID, "review", task)
|
||||||
|
|
||||||
if s.cfg.ExecutorFn == nil {
|
if s.cfg.ExecutorFn == nil {
|
||||||
return nil, fmt.Errorf("no executor configured")
|
return nil, fmt.Errorf("no executor configured")
|
||||||
}
|
}
|
||||||
|
t0 := time.Now()
|
||||||
result, err := s.cfg.ExecutorFn(ctx, iexec.Request{
|
result, err := s.cfg.ExecutorFn(ctx, iexec.Request{
|
||||||
SkillPrompt: s.cfg.SkillPrompt,
|
SkillPrompt: s.cfg.SkillPrompt,
|
||||||
TaskPrompt: task,
|
TaskPrompt: task,
|
||||||
@@ -58,24 +60,26 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if a.SessionID != "" && s.cfg.SessionsDir != "" {
|
||||||
|
_ = session.Append(s.cfg.SessionsDir, a.SessionID, session.Entry{
|
||||||
|
SessionID: a.SessionID,
|
||||||
|
Timestamp: time.Now(),
|
||||||
|
Skill: "review",
|
||||||
|
Phase: "review",
|
||||||
|
ProjectRoot: a.ProjectRoot,
|
||||||
|
Attempts: session.AttemptsFrom(result.Attempts),
|
||||||
|
FinalStatus: result.Status,
|
||||||
|
FilePath: result.FilePath,
|
||||||
|
ModelUsed: result.ModelUsed,
|
||||||
|
DurationMs: time.Since(t0).Milliseconds(),
|
||||||
|
Message: result.Message,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
b, err := json.Marshal(result)
|
b, err := json.Marshal(result)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, fmt.Errorf("marshal result: %w", err)
|
return nil, fmt.Errorf("marshal result: %w", err)
|
||||||
}
|
}
|
||||||
return b, nil
|
return b, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
func (s *Skill) prependHistory(sessionID, currentPhase, task string) string {
|
|
||||||
if sessionID == "" || s.cfg.SessionsDir == "" {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
entries, err := session.Read(s.cfg.SessionsDir, sessionID)
|
|
||||||
if err != nil || len(entries) == 0 {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
history := session.FormatHistory(entries, currentPhase)
|
|
||||||
if history == "" {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
return history + "\n---\n\n" + task
|
|
||||||
}
|
|
||||||
|
|||||||
@@ -5,6 +5,7 @@ import (
|
|||||||
"context"
|
"context"
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
"fmt"
|
"fmt"
|
||||||
|
"time"
|
||||||
|
|
||||||
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
||||||
"github.com/mathiasbq/supervisor/internal/session"
|
"github.com/mathiasbq/supervisor/internal/session"
|
||||||
@@ -48,11 +49,12 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
"phase: spec\nproject_root: %s\nrequirements: %s\noutput_path: %s\ncontext: %s\nmodel: %s",
|
"phase: spec\nproject_root: %s\nrequirements: %s\noutput_path: %s\ncontext: %s\nmodel: %s",
|
||||||
a.ProjectRoot, a.Requirements, outputPath, a.Context, model,
|
a.ProjectRoot, a.Requirements, outputPath, a.Context, model,
|
||||||
)
|
)
|
||||||
task = s.prependHistory(a.SessionID, "spec", task)
|
task = session.PrependHistory(s.cfg.SessionsDir, a.SessionID, "spec", task)
|
||||||
|
|
||||||
if s.cfg.ExecutorFn == nil {
|
if s.cfg.ExecutorFn == nil {
|
||||||
return nil, fmt.Errorf("no executor configured")
|
return nil, fmt.Errorf("no executor configured")
|
||||||
}
|
}
|
||||||
|
t0 := time.Now()
|
||||||
result, err := s.cfg.ExecutorFn(ctx, iexec.Request{
|
result, err := s.cfg.ExecutorFn(ctx, iexec.Request{
|
||||||
SkillPrompt: s.cfg.SkillPrompt,
|
SkillPrompt: s.cfg.SkillPrompt,
|
||||||
TaskPrompt: task,
|
TaskPrompt: task,
|
||||||
@@ -62,24 +64,26 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, err
|
return nil, err
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if a.SessionID != "" && s.cfg.SessionsDir != "" {
|
||||||
|
_ = session.Append(s.cfg.SessionsDir, a.SessionID, session.Entry{
|
||||||
|
SessionID: a.SessionID,
|
||||||
|
Timestamp: time.Now(),
|
||||||
|
Skill: "spec",
|
||||||
|
Phase: "spec",
|
||||||
|
ProjectRoot: a.ProjectRoot,
|
||||||
|
Attempts: session.AttemptsFrom(result.Attempts),
|
||||||
|
FinalStatus: result.Status,
|
||||||
|
FilePath: result.FilePath,
|
||||||
|
ModelUsed: result.ModelUsed,
|
||||||
|
DurationMs: time.Since(t0).Milliseconds(),
|
||||||
|
Message: result.Message,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
b, err := json.Marshal(result)
|
b, err := json.Marshal(result)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, fmt.Errorf("marshal result: %w", err)
|
return nil, fmt.Errorf("marshal result: %w", err)
|
||||||
}
|
}
|
||||||
return b, nil
|
return b, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
func (s *Skill) prependHistory(sessionID, currentPhase, task string) string {
|
|
||||||
if sessionID == "" || s.cfg.SessionsDir == "" {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
entries, err := session.Read(s.cfg.SessionsDir, sessionID)
|
|
||||||
if err != nil || len(entries) == 0 {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
history := session.FormatHistory(entries, currentPhase)
|
|
||||||
if history == "" {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
return history + "\n---\n\n" + task
|
|
||||||
}
|
|
||||||
|
|||||||
@@ -4,6 +4,7 @@ import (
|
|||||||
"context"
|
"context"
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
"fmt"
|
"fmt"
|
||||||
|
"time"
|
||||||
|
|
||||||
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
||||||
"github.com/mathiasbq/supervisor/internal/session"
|
"github.com/mathiasbq/supervisor/internal/session"
|
||||||
@@ -70,8 +71,15 @@ func (s *Skill) handleGreen(ctx context.Context, raw json.RawMessage) (json.RawM
|
|||||||
"phase: green\nproject_root: %s\ntest_path: %s\nmodel: %s\ntest_cmd: %s",
|
"phase: green\nproject_root: %s\ntest_path: %s\nmodel: %s\ntest_cmd: %s",
|
||||||
args.ProjectRoot, args.TestPath, s.resolveModel(args.Model), args.TestCmd,
|
args.ProjectRoot, args.TestPath, s.resolveModel(args.Model), args.TestCmd,
|
||||||
)
|
)
|
||||||
task = s.prependHistory(args.SessionID, "green", task)
|
task = session.PrependHistory(s.cfg.SessionsDir, args.SessionID, "green", task)
|
||||||
return s.execute(ctx, task)
|
|
||||||
|
t0 := time.Now()
|
||||||
|
result, err := s.execute(ctx, task)
|
||||||
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
|
}
|
||||||
|
s.logAttempt(args.SessionID, args.ProjectRoot, "tdd", "green", t0, result)
|
||||||
|
return result, nil
|
||||||
}
|
}
|
||||||
|
|
||||||
type refactorArgs struct {
|
type refactorArgs struct {
|
||||||
@@ -101,23 +109,15 @@ func (s *Skill) handleRefactor(ctx context.Context, raw json.RawMessage) (json.R
|
|||||||
"phase: refactor\nproject_root: %s\ntest_path: %s\nimpl_path: %s\nmodel: %s\ntest_cmd: %s",
|
"phase: refactor\nproject_root: %s\ntest_path: %s\nimpl_path: %s\nmodel: %s\ntest_cmd: %s",
|
||||||
args.ProjectRoot, args.TestPath, args.ImplPath, s.resolveModel(args.Model), args.TestCmd,
|
args.ProjectRoot, args.TestPath, args.ImplPath, s.resolveModel(args.Model), args.TestCmd,
|
||||||
)
|
)
|
||||||
task = s.prependHistory(args.SessionID, "refactor", task)
|
task = session.PrependHistory(s.cfg.SessionsDir, args.SessionID, "refactor", task)
|
||||||
return s.execute(ctx, task)
|
|
||||||
}
|
|
||||||
|
|
||||||
func (s *Skill) prependHistory(sessionID, currentPhase, task string) string {
|
t0 := time.Now()
|
||||||
if sessionID == "" || s.cfg.SessionsDir == "" {
|
result, err := s.execute(ctx, task)
|
||||||
return task
|
if err != nil {
|
||||||
|
return nil, err
|
||||||
}
|
}
|
||||||
entries, err := session.Read(s.cfg.SessionsDir, sessionID)
|
s.logAttempt(args.SessionID, args.ProjectRoot, "tdd", "refactor", t0, result)
|
||||||
if err != nil || len(entries) == 0 {
|
return result, nil
|
||||||
return task
|
|
||||||
}
|
|
||||||
history := session.FormatHistory(entries, currentPhase)
|
|
||||||
if history == "" {
|
|
||||||
return task
|
|
||||||
}
|
|
||||||
return history + "\n---\n\n" + task
|
|
||||||
}
|
}
|
||||||
|
|
||||||
func (s *Skill) resolveModel(override string) string {
|
func (s *Skill) resolveModel(override string) string {
|
||||||
@@ -127,6 +127,7 @@ func (s *Skill) resolveModel(override string) string {
|
|||||||
return s.cfg.DefaultModel
|
return s.cfg.DefaultModel
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// execute calls ExecutorFn and returns the marshaled result.
|
||||||
func (s *Skill) execute(ctx context.Context, task string) (json.RawMessage, error) {
|
func (s *Skill) execute(ctx context.Context, task string) (json.RawMessage, error) {
|
||||||
if s.cfg.ExecutorFn == nil {
|
if s.cfg.ExecutorFn == nil {
|
||||||
return nil, fmt.Errorf("no executor configured")
|
return nil, fmt.Errorf("no executor configured")
|
||||||
@@ -141,3 +142,28 @@ func (s *Skill) execute(ctx context.Context, task string) (json.RawMessage, erro
|
|||||||
}
|
}
|
||||||
return json.Marshal(result)
|
return json.Marshal(result)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// logAttempt writes a session.Entry for a completed phase if session_id is set.
|
||||||
|
// raw is the marshaled Result returned by execute; we unmarshal to extract fields.
|
||||||
|
func (s *Skill) logAttempt(sessionID, projectRoot, skill, phase string, t0 time.Time, raw json.RawMessage) {
|
||||||
|
if sessionID == "" || s.cfg.SessionsDir == "" {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
var result iexec.Result
|
||||||
|
if err := json.Unmarshal(raw, &result); err != nil {
|
||||||
|
return
|
||||||
|
}
|
||||||
|
_ = session.Append(s.cfg.SessionsDir, sessionID, session.Entry{
|
||||||
|
SessionID: sessionID,
|
||||||
|
Timestamp: time.Now(),
|
||||||
|
Skill: skill,
|
||||||
|
Phase: phase,
|
||||||
|
ProjectRoot: projectRoot,
|
||||||
|
Attempts: session.AttemptsFrom(result.Attempts),
|
||||||
|
FinalStatus: result.Status,
|
||||||
|
FilePath: result.FilePath,
|
||||||
|
ModelUsed: result.ModelUsed,
|
||||||
|
DurationMs: time.Since(t0).Milliseconds(),
|
||||||
|
Message: result.Message,
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|||||||
@@ -5,6 +5,7 @@ import (
|
|||||||
"context"
|
"context"
|
||||||
"encoding/json"
|
"encoding/json"
|
||||||
"fmt"
|
"fmt"
|
||||||
|
"time"
|
||||||
|
|
||||||
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
iexec "github.com/mathiasbq/supervisor/internal/exec"
|
||||||
"github.com/mathiasbq/supervisor/internal/session"
|
"github.com/mathiasbq/supervisor/internal/session"
|
||||||
@@ -58,6 +59,7 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
}
|
}
|
||||||
|
|
||||||
// ── Step 2: Writer agent (receives reader candidates) ────────────────────
|
// ── Step 2: Writer agent (receives reader candidates) ────────────────────
|
||||||
|
t0 := time.Now()
|
||||||
writerTask := fmt.Sprintf(
|
writerTask := fmt.Sprintf(
|
||||||
"role: writer\nsession_id: %s\nbrain_dir: %s\n\nreader_summary: %s\nreader_candidates:\n%s",
|
"role: writer\nsession_id: %s\nbrain_dir: %s\n\nreader_summary: %s\nreader_candidates:\n%s",
|
||||||
a.SessionID, s.cfg.BrainDir, readerResult.Message, readerResult.RunnerOutput,
|
a.SessionID, s.cfg.BrainDir, readerResult.Message, readerResult.RunnerOutput,
|
||||||
@@ -72,6 +74,18 @@ func (s *Skill) Handle(ctx context.Context, tool string, args json.RawMessage) (
|
|||||||
return nil, fmt.Errorf("writer agent: %w", err)
|
return nil, fmt.Errorf("writer agent: %w", err)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
_ = session.Append(s.cfg.SessionsDir, a.SessionID, session.Entry{
|
||||||
|
SessionID: a.SessionID,
|
||||||
|
Timestamp: time.Now(),
|
||||||
|
Skill: "trainer",
|
||||||
|
Phase: "trainer",
|
||||||
|
Attempts: session.AttemptsFrom(writerResult.Attempts),
|
||||||
|
FinalStatus: writerResult.Status,
|
||||||
|
ModelUsed: writerResult.ModelUsed,
|
||||||
|
DurationMs: time.Since(t0).Milliseconds(),
|
||||||
|
Message: writerResult.Message,
|
||||||
|
})
|
||||||
|
|
||||||
b, err := json.Marshal(writerResult)
|
b, err := json.Marshal(writerResult)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, fmt.Errorf("marshal result: %w", err)
|
return nil, fmt.Errorf("marshal result: %w", err)
|
||||||
|
|||||||
Reference in New Issue
Block a user