Skip to content

chore(deploy): the AI service project deploys only from main (#104) - #106

Draft
Fluory wants to merge 2 commits into
claude/fix-ai-build-skip-98from
claude/chore-ai-no-previews-104
Draft

Fluory wants to merge 2 commits into
claude/fix-ai-build-skip-98from
claude/chore-ai-no-previews-104

Conversation

@Fluory

@Fluory Fluory commented Oct 2, 2026 •

Copy link
Copy Markdown
Owner

Warum

Fixes #104 · Befund aus dem Review von #102 (#98): Auch mit dem „Ignored Build Step“ erzeugt jeder Push auf jeden PR-Branch ein Deployment des KI-Projekts requestflow-ai. Übersprungene Builds enden als CANCELED und zählen womöglich trotzdem gegen die Deploy-Grenzen des Hobby-Plans (am 2026-09-28: „Deployment rate limited – retry in 24 hours“). Gestapelt auf #102 (dieselbe Datei); nach dessen Merge Basis auf main.

Arbeitsstand

Was ist passiert (Klartext)

Der KI-Dienst hat bisher bei jedem Push auf irgendeinen Branch ein eigenes Vercel-Deployment angelegt – auch wenn er gar nicht gebaut wurde. Das verbraucht das begrenzte Deploy-Kontingent des kostenlosen Plans. Jetzt deployt das KI-Projekt nur noch von main. Vorschau-Versionen des KI-Dienstes braucht niemand: Web-Vorschauen rufen den KI-Dienst nicht auf, und die CI baut und testet das Image bei jeder Änderung daran. Ein Redeploy aus dem Dashboard (z. B. nach einer geänderten Einstellung) bleibt möglich.

Das ist eine Änderung an der Deploy-Konfiguration und braucht deine Freigabe vor dem Merge.

Plan-Pflicht (SYSTEM.md §4)

  • Kein Auslöser – keine Modulgrenze, öffentliche API, Migration, Auth/Rechte, kein Zahlungs-/Daten-/Infrapfad, höchstens zwei Module, keine Architekturvarianten, umkehrbar
  • Auslöser zutreffend – Impact Manifest ausgefüllt (Plan vor Code)

Hinweis: Deploy-Konfiguration (Infra-Regel: menschliche Freigabe), aber kein neuer Infrapfad und per Revert umkehrbar.

Geändert

Nachweis (SYSTEM.md §11)

  • Vercel-Doku (/docs/project-configuration/git-configuration, Stand 2026-08-25): Objekt Branch-Muster → Boolean; nicht genannte Branches sind true; minimatch-Muster; trifft ein Branch mehrere Regeln und eine davon ist true, wird deployt; die Einstellung betrifft nur Deployments durch Commits.
  • Lokal mit minimatch 5 nachgestellt: main → deployt; claude/fix-ai-build-skip-98, dependabot/npm_and_yarn/x, feature, main-old → nicht. (* würde Namen mit / nicht erfassen – deshalb **.)
  • verify:changed: scripts/doku-check.sh grün, JSON gültig
  • verify: CI am PR (keine Code-Änderung)
  • verify:full / E2E-Spec: nicht betroffen
  • Manueller Prüfnachweis: nach dem Merge – der nächste PR-Push zeigt kein requestflow-ai-Deployment; ein Merge auf main mit Änderung unter services/ai/ deployt Produktion.
  • Frischer Review (P1 vor Ready-for-review; Architektur/API/DB immer): chore(deploy): the AI service project deploys only from main (#104) #106 (comment) – nichts blockierend, beide should umgesetzt

Doku-Entscheidung (genau eine)

  • Keine langlebige Doku betroffen – Begründung: –
  • Doku betroffen und im selben PR aktualisiert:

Entferntes oder Umbenanntes: docs/ + README gegrept, Treffer bereinigt: Hinweis „AI preview may be missing“ in §3.6 ersetzt

Dateigrößen und neue Bausteine (SYSTEM.md §7)

Dateien über 500 Zeilen im Diff (Ausnahmen: generierter Code, Lockfiles, Fixtures, Migrationen, Schemas, Ressourcen, Doku, Konfiguration):

  • keine
  • bewusst belassen – Begründung: –
  • im selben PR nach fachlicher Verantwortung geteilt
  • Folge-Issue –

Über 800 Zeilen mit neuer Fachlogik oder über 1000 Zeilen (P1/P2): nicht betroffen

Neue Shared-Komponente, Utility-Datei, Adapter oder fachlicher Service:

  • nein
  • ja – gesucht nach: –

Subagent-Einsätze

  • 1 × frischer Review (read-only, Sonnet) – nichts blockierend, 2 should + 3 nits; Umgang im PR-Kommentar.

Risiken / offene Punkte

🤖 Generated with Claude Code

https://claude.ai/code/session_01DJ5vaKvTYiMvdngT4d3xo1

git.deploymentEnabled { "**": false, "main": true } – PR branches create
no requestflow-ai deployment, so they stop counting against the Hobby
deployment limits. CI keeps building and smoke-testing the image (#79).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DJ5vaKvTYiMvdngT4d3xo1
@vercel

vercel Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
requestflow Ready Ready Preview Oct 2, 2026 8:24am UTC

Fluory commented Oct 2, 2026

Copy link
Copy Markdown
Owner Author

Frischer Review (read-only Subagent) – Ergebnis und Umgang

Nichts blockierend. Bestätigt: ** erfasst Branches mit / (claude/…, dependabot/…); main trifft ** (false) und main (true) → deployt; das Web-Projekt liest die Root-vercel.json ohne git-Block und ist nicht betroffen; die Einstellung sperrt nur Git-Push-Deployments, Redeploys aus dem Dashboard (§9) bleiben möglich.

Befund Umgang
should – „Web-Previews rufen den produktiven KI-Dienst“ widerspricht §4 (Preview-Variablen bleiben leer) behoben in 1cc5f68: „web previews do not call the AI service (§4)“
should – CI-Aussage zu #79 ungenau behoben: „CI builds the image and checks /healthz on AI-image changes (#79)“
nit – überlange Zeile, „see 7“ behoben
nit – Preview-Schutz in Schritt 2 jetzt ohne Wirkung ergänzt: gilt nur, falls Git-Deployments anderer Branches wieder eingeschaltet werden
nit – „skipped builds may still count“ in Schritt 6 für PRs nun weitgehend gegenstandslos bleibt: gilt weiter für main-Pushes ohne KI-Änderung

scripts/doku-check.sh grün, JSON gültig.


Generated by Claude Code

This branch was successfully deployed

1 active deployment
Preview – requestflow — 1cc5f687 Deployed Oct 2, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants