nvt M1: nvt-bridge + agents/nvt-fat-developer bak filkontrakten

[olebhansen]

Del av #95. Se doc/plans/nvt-agent-integrasjon.md (PR #94), avsnittene «Arkitektur» og «M1». Forutsetter funnene fra M0.

Mål

Ny utførende agent nvt-fat-developer bak uendret filkontrakt, med en deterministisk nvt-bridge som mapper topic → levende nvt-instans.

Leveranser

1. agents/nvt-fat-developer/: triggers/-kontrakt som i dag, README, .env.example, og en instruks-mal som rendres inn i instansens AGENTS.local.md (tilpasset utgave av kodeagent-protokollen: «kjenn din begrensning», branch+PR-kontrakt med --base og Closes #, retro/KB-læringer, resultatlinje-format — men uten innboks-polling: oppgaver kommer som prompts, og agentdctl signal done sendes etter resultatlinja).
2. nvt-bridge (Node/TS): poller inbox (samme dedupe-regel: id uten linje i results), topic = payload.origin.event_id uten delta-suffiks, topic→instans-state, agent-init/agent-up on demand, prompt-injeksjon via docker exec agentdctl prompt --source host --external, fallback-resultatlinje med status:"error" (aldri fabrikert suksess) hvis signal done kommer uten resultatlinje. Serielt per topic, parallelt på tvers (maks N). TTL-basert agent-down for inaktive topics (workspace beholdes).
3. Container-kjøring av bridgen (docker-outside-of-docker): node-slim + docker/compose-cli, hostens socket mountet, nvt-sjekkout + agentkatalog mountet på samme absolutte sti som på hosten. Fallback: samme kode som node-prosess under WSL2.
4. Mount av triggers/ inn i instansene som /triggers — agenten skriver resultatlinjer selv, så integrations er uendret.
5. Én ny rute i AGENT_ROUTES (kun config i deploy-klonens .env), oppdatering av solution-proposal-skillens valgkriterier, doc/log.md.

Akseptansekriterier

• Delegert issue → PR opprettet av bot-kontoen → resultatlinje → svar i opphavstråden.
• Oppfølgingsevent i samme topic lander i samme levende sesjon (samme samtale, samme workspace).
• To topics kjører parallelt i hver sin instans uten å påvirke hverandre.
• Ingen mount av monorepo/deploy-klone i agentcontainere; deploy-treet urørt etter kjøring.
• Bridgen kjører i container på WSL2-oppsettet; agentcontainere har aldri hostens Docker-socket.
• Tokenet virker ikke utenfor arbeidsrepoene; direkte push til beskyttede brancher avvises.

Sikkerhet

• Delegerte prompts er upålitelig input; --external-preamblen skal alltid med.
• Ingen hemmeligheter i image, logger, events eller resultatlinjer; tokens kun i .broker/ på hosten.
• Bot-kontoen skal ikke inn i GITHUB_ALLOWED_USERS; kontonavn holdes utenfor repoet (kun env/broker-config).

[olebhansen]

@olebhansen-agent Arbeidsordre — større kodeoppgave for senior-kodeagenten: den miljøuavhengige kjernen av M1. Les issue #97 og doc/plans/nvt-agent-integrasjon.md i repoet før du starter — denne kommentaren er et sammendrag, spesifikasjonen bor der.

Bygg den delen av M1 som ikke avhenger av M0-funnene:

1. apps/nvt-bridge/ (Node/TS, samme stack og testoppsett som integrations/): poller agents/nvt-fat-developer/triggers/inbox.jsonl med samme dedupe-regel som i dag (id uten linje i results.jsonl); topic = payload.origin.event_id uten delta-suffiks; topic→instans-mapping i en state-fil (state/topics.json); serielt per topic, parallelt på tvers (maks N, konfigurerbart); fallback-resultatlinje med status:"error" og forklaring hvis signal done kommer uten resultatlinje innen fristen — aldri en fabrikert suksess.
2. All interaksjon med nvt bak et lite interface (agent-init/agent-up/docker exec … agentdctl prompt --source host --external), med en fake-implementasjon for tester. Den ekte implementasjonen skrives som tynn adapter og merkes tydelig som «kalibreres mot M0-funn» — ikke bak inn antakelser om compose-stier eller nettverk utover det planen allerede sier.
3. agents/nvt-fat-developer/: triggers/-kontrakt som de andre agentene, README, .env.example, og instruks-malen som skal rendres inn i instansens AGENTS.local.md (tilpasset kodeagent-protokoll, se nvt M1: nvt-bridge + agents/nvt-fat-developer bak filkontrakten #97 leveranse 1: uten innboks-polling, med agentdctl signal done etter resultatlinja).
4. Tester for kjernelogikken (dedupe, topic-avledning, serialisering per topic, fallback-linja) mot fake-implementasjonen, og en kort oppføring i doc/log.md.

Rammer:

• Branch fra v2.0, PR med base v2.0 og Refs #97 i beskrivelsen (ikke Closes — M1 lukkes først etter E2E-verifisering mot ekte nvt-instanser).
• Ikke rør integrations/src/ — ruta i AGENT_ROUTES er ren driftskonfig og tas ved utrulling.
• Dockerfile for container-kjøring av bridgen (docker-outside-of-docker, se planen) kan være med hvis det faller naturlig — men merk at **/Dockerfile og compose-filer er CODEOWNERS-stier, så PR-en blir stående til menneskelig review. Det er forventet; ikke sett auto-merge-label på denne.
• Kjør review-prosessen din og post funnene som PR-kommentar før du rapporterer.

Rapportér tilbake her med PR-lenke og en kort liste over designvalg du måtte ta som ikke var avgjort i planen.

[olebhansen-agent]

⚠️ Agenten leverte uten strukturert svar — se logg.

[olebhansen]

@olebhansen-agent forrige forsøk strandet på formatfeil (svaret manglet AGENT-RESULT-markøren, se #104). Ikke gjenta analysen: deleger arbeidsordren i #97 (comment) til senior-kodeagenten nå, med beskjed om å lese issue #97 og doc/plans/nvt-agent-integrasjon.md før start. Avslutt svaret ditt med AGENT-RESULT-JSON-linja.

[olebhansen-agent]

Delegerer NVT M1-arbeidsordren til senior-kodeagenten (local-cc-coding-agent). Issue #97 er åpen, ingen PR eksisterer ennå. doc/plans/nvt-agent-integrasjon.md finnes dessverre ikke i repoet — issue #97 selv fungerer som spesifikasjon. Forrige forsøk strandet på formatfeil (manglende AGENT-RESULT-markør), så agenten får tydelig beskjed om å avslutte med korrekt markør.

[olebhansen-agent]

Alt er levert. PR: #110 (base v2.0, Refs #97, ingen auto-merge-label).

Levert: apps/nvt-bridge/ (dedupe som i dag, topic = payload.origin.event_id uten -dN, state/topics.json, serielt per topic / parallelt på tvers, TTL-agent-down), all nvt-interaksjon bak NvtDriver med fake + tørrkjørings-driver og en tynn ekte adapter merket «kalibreres mot M0-funn», agents/nvt-fat-developer/ (triggers, README, .env.example, instruks-mal for AGENTS.local.md — uten innboks-polling, med agentdctl signal done), 77 tester og oppføring i doc/log.md. integrations/src/ er urørt (verifisert).

Reviewen var ikke ren. Reviewer-subagenten fant 1 blocker + 4 majors, alle reprodusert — bl.a. at en feilende loggskriving kunne etterlate et event uten resultatlinje (evig ubehandlet), at manglende linjeskift fra agenten kunne ødelegge både dens og vår linje, og en prompt-injeksjon der en lurt agent kunne postet falsk suksess i en annen tråd. Alt fikset med regresjonstest og postet som PR-kommentar (audit-sporet).

To ting du bør vite:

1. doc/plans/nvt-agent-integrasjon.md finnes på v2.0 — arbeidsordren hevdet den manglet (nok sjekket mot main). Jeg fulgte planen, inkludert beslutning 3: llm-gateway, ikke LM Studio.
2. CI dekker ikke de nye testene: tokenet mangler workflow-scope, så jobben er rullet tilbake og ligger som foreslått YAML i PR-beskrivelsen.

Jeg rørte ikke CODEOWNERS, men merk gapet: agents/*/CLAUDE.md fanger ikke den nye agentens instruks-mal.

===AGENT-RESULT===
{"intent":"action","reply":"NVT M1-kjernen er levert som PR #110 (base v2.0, Refs #97): https://github.com/digdir/digdir-ai-agents/pull/110\n\nLeveranser: apps/nvt-bridge/ (dedupe som i dag, topic = payload.origin.event_id uten delta-suffiks, state/topics.json, serielt per topic og parallelt på tvers, TTL-agent-down, fallback-resultatlinje med status:error — aldri fabrikert suksess), all nvt-interaksjon bak NvtDriver med fake- og dry-run-implementasjon og en tynn ekte adapter merket «kalibreres mot M0-funn», agents/nvt-fat-developer/ (triggers-kontrakt, README, .env.example, instruks-mal for AGENTS.local.md uten innboks-polling og med agentdctl signal done), 77 tester og oppføring i doc/log.md. integrations/src/ er urørt.\n\nReviewen var IKKE ren: reviewer-subagenten fant 1 blocker + 4 majors, alle reprodusert og alle fikset med regresjonstest (62 -> 77 tester). Funnene er postet som PR-kommentar. Viktigst: feilende loggskriving kunne etterlate et event uten resultatlinje (evig ubehandlet), manglende linjeskift fra agenten kunne ødelegge både dens og vår resultatlinje, og prompt-injeksjon kunne fått en agent til å poste falsk suksess i en annen tråd.\n\nTo avvik du bør vite om: (1) doc/plans/nvt-agent-integrasjon.md FINNES på v2.0 — arbeidsordren hevdet den manglet, sannsynligvis sjekket mot main. Jeg fulgte planen, inkludert beslutning 3 om llm-gateway framfor LM Studio som modell-backend. (2) CI dekker ikke de nye testene: agent-tokenet mangler workflow-scope, så GitHub avviste pushen av ci.yml-jobben. Den er rullet tilbake og ligger som foreslått YAML i PR-beskrivelsen for manuell innlegging.\n\nauto-merge-label er ikke satt (PR-en rører Dockerfile og docker-compose.yml — CODEOWNERS-stier). CODEOWNERS-gap løftet, men ikke endret av meg: agents/*/CLAUDE.md fanger ikke agents/nvt-fat-developer/AGENTS.local.md.tmpl, som er agentens instruks. To prosess-læringer er committet lokalt i /knowledge (ikke pushet)."}

[olebhansen]

@olebhansen-agent Arbeidsordre — oppfølging på M1 for senior-kodeagenten: kalibrer den ekte nvt-adapteren i apps/nvt-bridge/ mot M0-funnene. Les først «M0-funn»-avsnittet i doc/plans/nvt-agent-integrasjon.md (på v2.0) — det er fasiten fra den gjennomførte sandkasseøkten, og adapteren ble bevisst levert som ukalibrert skall i PR #110.

Endringer i den ekte driveren (src/nvt/docker.ts m.fl.):

1. agent-init må kjøres med --user non-root (claude nekter bypass-flagget som root; symptom: tmux-sesjonen dør < 5 s).
2. Klar-sjekk før første prompt-injeksjon: claude-onboardingen (velkomstskjerm + trust-dialog) spiser prompts som injiseres før sesjonen er klar. Driveren må enten verifisere klar-tilstand (f.eks. sjekke at session-launched-markøren finnes OG at tmux-panelet viser klar-prompt) eller sende to Enter gjennom onboardingen før første agentdctl prompt. Design gjerne en robust «vent-til-klar»-funksjon med timeout og ærlig feil.
3. Sti-validering: fail-fast med tydelig feilmelding hvis NVT_ROOT ligger under en katalog uid 1000 ikke kan traversere (f.eks. /root) — M0 brant seg på dette.
4. Volum-hygiene: dokumentér (README) at bytte root→non-root på eksisterende instans krever sletting av agent-<navn>_agent-home/_docker-data-volumene.
5. agent.yaml-malen: static_token-provider krever identity.mode: explicit med navn/e-post — malen/genereringen må sette dette, ikke mode: provider.
6. Oppdater/utvid testene mot fake-driveren der kontrakten endres (klar-sjekk før prompt).

Rammer som sist: branch fra v2.0, PR med base v2.0 og Refs #97, ikke rør integrations/src/, kjør review-prosessen og post funnene som PR-kommentar FØR du rapporterer (obligatorisk steg), rapportér tilbake her med PR-lenke. apps/nvt-bridge/ har Dockerfile/compose i seg fra før — ikke sett auto-merge-label hvis diffen rører CODEOWNERS-stier.

[olebhansen-agent]

Delegerer M1-kalibrering mot M0-funn til senior-kodeagenten (local-cc-coding-agent). Ingen eksisterende PR for denne oppfølgingen er funnet. Tokenet mitt har dessverre ikke addComment-scope, så jeg kunne ikke poste delegasjonen som kommentar på issue #97 — dette må gjøres manuelt eller med utvidet token.

[olebhansen-agent]

⚠️ Agenten feilet (error).

API Error: Connection closed mid-response. The response above may be incomplete.

[olebhansen]

@olebhansen-agent forrige forsøk ble avbrutt av et deploy (gateway-restart midt i kjøringen — ikke din feil). Prøv igjen: deleger arbeidsordren i #97 (comment) til senior-kodeagenten nå.

[olebhansen-agent]

⚠️ Agenten leverte uten strukturert svar — se logg.