OSVauco — Tech Review rapport

Perspektiv: Senior Google Cloud Platform engineer
Dato: 2026-05-26 06:03 CEST
Kilde: Perplexity OPAX-agent, basert på MASTERPLAN.md + repo-struktur + sesjonsfiler

✅ Det som er solid

CI/CD er riktig bygget. Cloud Build 2nd gen + GitHub App er den anbefalte veien — unngår OAuth-kompleksiteten i 1st gen.
Light/heavy LLM-ruting er en fornuftig kostnadsarkitektur. Flash foran, Pro ved behov — dette skalerer.
Teardown-disiplin (daglig 03-teardown.sh) viser forståelse for at Agent Runtimes og Reasoning Engines koster 24/7 selv uten trafikk.
MASTERPLAN.md som single source of truth holder agenter og mennesker synkronisert.

⚠️ Kritiske strukturproblemer

1. Repo-identitet er uklar

Repo	Hva det sier det er	Hva det faktisk er	Konflikt?
`OSVauco`	Regler, GCP-scripts, agenter	Alt — inkl. `main.py`, `ml/`, `agents/`, `infrastructure/`, `protocols/`	Ja — monorepo uten grenser
`vauco-gemini-tui-bridge`	TUI-cockpit	Uklart om dette er en separat service eller shell-wrapper	Ukjent
`vauco-os`	Det kundene bruker	Ukjent om dette er fersk kopi av OSVauco eller eget produkt	Potensielt duplikat
`vauco-bootstrap`	Onboarding	Ingen Terraform ennå — grense mot `OSVauco/infrastructure/` er uklar	Uklar
`deep-dream`	Strategisk visjon	"Aldri deploy herfra" — er dette et repo eller et dokument?	Unødvendig repo

Problemet: OSVauco er blitt et monorepo som inneholder operasjonslogikk, ML-kode, infrastruktur-scripts, agent-kode, docs og protokoller. For mye i én kasse.

2. `.env.prefilled` i repoet er en sikkerhetsrisiko

.env.prefilled i root kan inneholde faktiske project IDs og account-verdier. For en SaaS-plattform med kunder bør dette enten fjernes eller eksplisitt gitignores.

Aksjon: Verifiser innhold → fjern eller legg til .gitignore.

3. `Procfile` i root antyder Heroku-arv

Procfile er et Heroku-konsept. Cloud Run bruker container entrypoint, ikke Procfile. Dette er enten teknisk gjeld eller det skaper forvirring om hva som faktisk starter appen.

Aksjon: Fjern eller dokumenter tydelig at den er ubrukt.

4. `cloudbuild.base.yaml` + `cloudbuild.yaml` — uklar relasjon

To Cloud Build-filer i root uten åpenbar arv/include-struktur. Forvirrende for agenter og nye devs — hvilken er master?

Aksjon: Slå sammen eller dokumenter arv eksplisitt i begge filer.

5. Infrastructure scripts vs Terraform — ikke-idempotent overgangsperiode

Terraform-migreringen (Fase C) skaper en farlig periode der begge tilstandene eksisterer. Ingen lock hindrer at noen kjører et shell-script etter at Terraform har overtatt samme ressurs → state-konflikt.

Aksjon: Definer eksplisitt deprecerings-plan for hvert script etter Terraform-migrering.

🏗️ Anbefalt repo-struktur (mål, ikke krav nå)

# I dag (problem):
OSVauco/              ← monorepo: alt blandet

# Anbefalt (Fase C-mål):
vauco-platform/
  ├── services/osvauco-agent/   ← main.py + agents/ + ml/
  ├── infrastructure/terraform/ ← .tf-filer
  ├── infrastructure/scripts/   ← shell (avvikles gradvis)
  └── dialogflow/               ← B6 her

vauco-ops/            ← internt operasjonsverktøy
  ├── protocols/
  ├── docs/
  └── .gemini/

vauco-bootstrap/      ← Terraform-modul per kunde

Merk: Omstrukturering er en Fase C-vurdering — ikke krav nå.

📋 Prioritert aksjonsliste

Prioritet	Handling	Innsats	Risiko hvis ignorert
🔴 1	Sjekk `.env.prefilled` — fjern eller gitignore	5 min	Sensitiv info i Git
🔴 2	Avklar rolle til `vauco-os` vs `OSVauco` — duplikat?	30 min	Feil repo ved kundelevering
🟡 3	Fjern/rename `Procfile` eller dokumenter at den er ubrukt	5 min	Forvirrer agenter og nye devs
🟡 4	Slå sammen `cloudbuild.base.yaml` og `cloudbuild.yaml`	15 min	CI/CD-feil ved endringer
🟡 5	Legg til `infrastructure/terraform/README.md` nå	10 min	Ingen klar destinasjon for Terraform-bootstrap
🟢 6	Vurder om `deep-dream` trenger eget repo	10 min	Unødvendig repo-overhead
🟢 7	Plan for deprecering av shell-scripts etter Terraform	Fase C	State-konflikt sh vs tf

🎯 Konklusjon

Planen er arkitektonisk fornuftig og GCP-nativ. Fase A og B er godt gjennomført. De største risikoene er ikke tekniske — de er organisatoriske:

Repo-grensene er uklare
Monorepo-vekst er i gang uten styring
Overgangen fra shell til Terraform mangler eksplisitt låsestrategi

Adresser de 🔴 røde punktene før B6 og Terraform-bootstrap for å stå stødigere til Fase C.

Generert av Perplexity OPAX-agent | propane-will-491900-m5 | 2026-05-26 06:03 CEST

4.8 KiB Raw Blame History