Skip to content

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

Merged
Fluory merged 5 commits into
mainfrom
claude/chore-ai-no-previews-104
Oct 10, 2026
Merged

Fluory merged 5 commits into
mainfrom
claude/chore-ai-no-previews-104

Conversation

@Fluory

@Fluory Fluory commented Oct 9, 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“). Ersetzt #106 (geschlossen, weil der Basis-Branch von #102 nach dessen Merge gelöscht wurde); #102 ist in main, main ist eingemergt.

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: grün – CI check Run 38006317015 am PR-Kopf 28b1177 (Doku + JSON, 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

Fluory and others added 5 commits September 28, 2026 22:24
…nged (#98)

Every push – main and every PR – built and pushed a new AI image, also for
web-only changes, until the Hobby container registry hit its image limit and
the production build of #95 failed. An ignoreCommand now skips the build
unless services/ai changed since the last successful deployment (else the
parent commit; if git cannot compare, it builds). The runbook names the
registry limit and how to prune it.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…nest scope

- a redeploy runs the Ignored Build Step too: after rotating a secret or the
  Gemini switch, redeploy without it and check Ready (CANCELED = not active)
- skipped builds may still count against deployment limits: the CHANGELOG no
  longer promises fewer deployments
- first build of a new branch compares only its last commit (AI preview may
  be missing); vcr commands name --project

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
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
…eviews-104

# Conflicts:
#	CHANGELOG.md
#	docs/technical/deployment-vercel.md
#	services/ai/vercel.json
@vercel

vercel Bot commented Oct 9, 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 9, 2026 11:53pm UTC

@Fluory
Fluory marked this pull request as ready for review October 9, 2026 23:56
@Fluory
Fluory merged commit ac763dc into main Oct 10, 2026
6 of 8 checks passed

This branch was successfully deployed

1 active deployment
Preview – requestflow — 28b11776 Deployed Oct 9, 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.

infra(vercel): no preview deployments for the AI service project

2 participants