@contentful/experience-design-system-cli 2.12.2-dev-build-e72ea10.0 → 2.12.3-dev-build-4a6b002.0

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.12.2-dev-build-e72ea10.0",
+  "version": "2.12.3-dev-build-4a6b002.0",
   "description": "Contentful Experiences design system import CLI",
   "license": "MIT",
   "type": "module",

package/dist/src/import/tui/WizardApp.js

@@ -49,7 +49,7 @@ import { checkAgentAuth } from '../../generate/agent-runner.js';
 import { normalizePath } from '../path-utils.js';
 import { DEFAULT_CONFIGURED_HOST, toConfiguredHost } from '../../host-utils.js';
 import { writeExperiencesCredentials } from '../../credentials-store.js';
-import { nextStepAfterScopeGate, nextStepAfterCredentialsValidated, shouldBypassPreview, buildSkippedPreviewTransition, shouldRefusePush, buildSkippedPushTransition, resolveNoCacheForGenerate, } from './wizard-state-transitions.js';
+import { nextStepAfterScopeGate, nextStepAfterCredentialsValidated, shouldBypassPreview, buildSkippedPreviewTransition, shouldRefusePush, buildSkippedPushTransition, shouldSkipFinalReviewAfterCredentials, resolveNoCacheForGenerate, } from './wizard-state-transitions.js';
 function findCliPath() {
     return join(fileURLToPath(import.meta.url), '..', '..', '..', '..', '..', 'bin', 'cli.js');
 }
@@ -197,11 +197,6 @@ export function WizardApp({ initialSpaceId = '', initialEnvironmentId = 'master'
     // `generating-tokens` step which already drives the token-classification
     // subprocess off `state.rawTokensPath`. Modify-entry wins if both are set.
     const rawTokensEntryReady = !modifyEntryReady && !pushFromPickerReady && !!initialRawTokensPath;
-    // Fresh session = the wizard is going to create its own extract session via
-    // `analyze extract` rather than pick one up from `--modify` / `--push-from-run`.
-    // For fresh sessions we force `--no-cache` on the spawned `generate components`
-    // so operators aren't silently served classifications cached from a prior
-    // session (the generation_cache table is project-wide, not session-scoped).
     const isFreshSession = !seedExtractSessionId;
     const effectiveNoCache = resolveNoCacheForGenerate({ isFreshSession, cliNoCache: noCache });
     const initialStepResolved = modifyEntryReady
@@ -270,6 +265,7 @@ export function WizardApp({ initialSpaceId = '', initialEnvironmentId = 'master'
         credentialsSkipped: false,
         lastRunId: null,
         finalizeErrorBanner: null,
+        finalReviewPassed: modifyEntryReady || pushFromPickerReady,
     });
     useEffect(() => {
         sessionRef.current = {
@@ -813,10 +809,7 @@ export function WizardApp({ initialSpaceId = '', initialEnvironmentId = 'master'
     // accepted components) or jumps straight to push-decision-gate (if scope
     // rejected everything but tokens/removals still need to be pushed).
     const advanceAfterCredentialsValidated = async () => {
-        // If we already ran the generator (re-entering credentials from a late
-        // 401/403 raised by runPreview), skip back to push-decision-gate rather
-        // than re-running the LLM.
-        if (state.generateSessionId) {
+        if (shouldSkipFinalReviewAfterCredentials(state)) {
             update({ step: 'push-decision-gate' });
             return;
         }
@@ -1511,6 +1504,7 @@ export function WizardApp({ initialSpaceId = '', initialEnvironmentId = 'master'
             case 'final-review': {
                 return (_jsx(FinalReviewHost, { extractSessionId: state.extractSessionId, generatedCount: state.generatedCount, autoAccept: autoAcceptScope, livePreview: livePreview, spaceId: state.spaceId, environmentId: state.environmentId, cmaToken: state.cmaToken, host: state.host, tokensPath: state.tokensPath, initialFinalizeError: state.finalizeErrorBanner, onFinalize: (accepted, rejected, unresolved) => {
                         process.stderr.write(`Accepted: ${accepted}  Rejected: ${rejected}  Unresolved: ${unresolved}\n`);
+                        update({ finalReviewPassed: true });
                         // INTEG-4411 refined: no `accepted === 0` up-front block here.
                         // A zero-accepted finalize can still be a valid push when the
                         // operator explicitly rejected component(s) that exist server-

package/dist/src/import/tui/wizard-state-transitions.d.ts

@@ -43,21 +43,10 @@ export declare function nextStepAfterScopeGate(opts: {
 export declare function nextStepAfterCredentialsValidated(opts: {
     acceptedCount: number;
 }): WizardStepAfterCredentials;
-/**
- * Bug fix (2026-07): the `generation_cache` table in pipeline.db is keyed by
- * `(input_hash, prompt_hash, entity_type, component_id)` — with no session
- * scoping. `component_id` is a deterministic hash of `(name, source)`, so a
- * fresh new session on an unchanged codebase always hits the cache and
- * silently skips the LLM. That's the intended behavior for continued sessions
- * (modify / push-from-run) but is surprising for a fresh `experiences import`
- * run: the operator expects a re-classify pass, not cached output from a
- * prior session.
- *
- * Resolution: for a fresh session (no `seedExtractSessionId`), force
- * `--no-cache` on the spawned `generate components` subprocess. For a
- * continued session (modify / push-from-run / picker-seeded), honor the
- * operator's `--no-cache` flag as before.
- */
+export declare function shouldSkipFinalReviewAfterCredentials(state: {
+    generateSessionId: string | null;
+    finalReviewPassed: boolean;
+}): boolean;
 export declare function resolveNoCacheForGenerate(opts: {
     isFreshSession: boolean;
     cliNoCache: boolean;

package/dist/src/import/tui/wizard-state-transitions.js

@@ -43,21 +43,9 @@ export function nextStepAfterScopeGate(opts) {
 export function nextStepAfterCredentialsValidated(opts) {
     return opts.acceptedCount > 0 ? 'generating' : 'push-decision-gate';
 }
-/**
- * Bug fix (2026-07): the `generation_cache` table in pipeline.db is keyed by
- * `(input_hash, prompt_hash, entity_type, component_id)` — with no session
- * scoping. `component_id` is a deterministic hash of `(name, source)`, so a
- * fresh new session on an unchanged codebase always hits the cache and
- * silently skips the LLM. That's the intended behavior for continued sessions
- * (modify / push-from-run) but is surprising for a fresh `experiences import`
- * run: the operator expects a re-classify pass, not cached output from a
- * prior session.
- *
- * Resolution: for a fresh session (no `seedExtractSessionId`), force
- * `--no-cache` on the spawned `generate components` subprocess. For a
- * continued session (modify / push-from-run / picker-seeded), honor the
- * operator's `--no-cache` flag as before.
- */
+export function shouldSkipFinalReviewAfterCredentials(state) {
+    return state.finalReviewPassed && state.generateSessionId != null;
+}
 export function resolveNoCacheForGenerate(opts) {
     if (opts.isFreshSession)
         return true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.12.2-dev-build-e72ea10.0",
+  "version": "2.12.3-dev-build-4a6b002.0",
   "description": "Contentful Experiences design system import CLI",
   "license": "MIT",
   "type": "module",
@@ -37,7 +37,7 @@
     "svelte": "^5.56.4",
     "ts-morph": "^27.0.2",
     "typescript": "^5.9.3",
-    "@contentful/experience-design-system-types": "2.12.2-dev-build-e72ea10.0"
+    "@contentful/experience-design-system-types": "2.12.3-dev-build-4a6b002.0"
   },
   "devDependencies": {
     "@tsconfig/node24": "^24.0.3",