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

package/dist/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.12.2-dev-build-4dc32df.0",
+  "version": "2.12.2-dev-build-e72ea10.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, shouldSkipFinalReviewAfterCredentials, } from './wizard-state-transitions.js';
+import { nextStepAfterScopeGate, nextStepAfterCredentialsValidated, shouldBypassPreview, buildSkippedPreviewTransition, shouldRefusePush, buildSkippedPushTransition, resolveNoCacheForGenerate, } from './wizard-state-transitions.js';
 function findCliPath() {
     return join(fileURLToPath(import.meta.url), '..', '..', '..', '..', '..', 'bin', 'cli.js');
 }
@@ -197,6 +197,13 @@ 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
         ? 'final-review'
         : pushFromPickerReady
@@ -263,7 +270,6 @@ export function WizardApp({ initialSpaceId = '', initialEnvironmentId = 'master'
         credentialsSkipped: false,
         lastRunId: null,
         finalizeErrorBanner: null,
-        finalReviewPassed: false,
     });
     useEffect(() => {
         sessionRef.current = {
@@ -607,7 +613,7 @@ export function WizardApp({ initialSpaceId = '', initialEnvironmentId = 'master'
                 tokensPath,
                 agent: state.agent,
                 ...(state.agentModel ? { model: state.agentModel } : {}),
-                noCache,
+                noCache: effectiveNoCache,
             }),
         ];
         let progressCursor = null;
@@ -682,7 +688,7 @@ export function WizardApp({ initialSpaceId = '', initialEnvironmentId = 'master'
                 tokensPath,
                 agent: state.agent,
                 ...(state.agentModel ? { model: state.agentModel } : {}),
-                noCache,
+                noCache: effectiveNoCache,
                 generatePromptPath,
             }),
         ];
@@ -807,15 +813,10 @@ 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 the operator already went through final-review once (re-entering
-        // credentials from a late 401/403 raised by runPreview), skip back to
-        // push-decision-gate rather than making them review again. We gate on
-        // `finalReviewPassed` specifically — NOT `generateSessionId` alone —
-        // because the scope-gate background prefetch also populates
-        // `generateSessionId` as soon as its LLM finishes, and if that lands
-        // while the operator is still typing credentials it would otherwise
-        // trip this shortcut and skip past `final-review` entirely.
-        if (shouldSkipFinalReviewAfterCredentials(state)) {
+        // 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) {
             update({ step: 'push-decision-gate' });
             return;
         }
@@ -1510,10 +1511,6 @@ 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`);
-                        // Mark the review as passed so a later re-entry to credentials
-                        // (e.g. late 401 raised by runPreview) short-circuits back to
-                        // push-decision-gate instead of re-rendering final-review.
-                        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

@@ -44,24 +44,23 @@ export declare function nextStepAfterCredentialsValidated(opts: {
     acceptedCount: number;
 }): WizardStepAfterCredentials;
 /**
- * Bug fix (2026-07): the "skip generator, jump to push-decision-gate" shortcut
- * on the credentials → post-credentials edge was originally guarded by
- * `state.generateSessionId != null`. That signal is ALSO set by the scope-gate
- * background prefetch as soon as the LLM finishes — so when a fast prefetch
- * completed while the operator was still typing credentials, submitting them
- * skipped `final-review` entirely and jumped straight to `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.
  *
- * The real invariant the guard wants is: has the operator already been through
- * the final-review screen and finalized once? That's what `finalReviewPassed`
- * captures — it flips true in `onFinalize` on the GenerateReviewStep, not on
- * mere prefetch completion.
- *
- * Retained (and required) is `generateSessionId != null`, so we don't jump to
- * push-decision-gate before generation is even complete.
+ * 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;
+export declare function resolveNoCacheForGenerate(opts: {
+    isFreshSession: boolean;
+    cliNoCache: boolean;
 }): boolean;
 /**
  * Skip-credentials spec — Task 2. When the operator pressed `s` on the

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

@@ -44,23 +44,24 @@ export function nextStepAfterCredentialsValidated(opts) {
     return opts.acceptedCount > 0 ? 'generating' : 'push-decision-gate';
 }
 /**
- * Bug fix (2026-07): the "skip generator, jump to push-decision-gate" shortcut
- * on the credentials → post-credentials edge was originally guarded by
- * `state.generateSessionId != null`. That signal is ALSO set by the scope-gate
- * background prefetch as soon as the LLM finishes — so when a fast prefetch
- * completed while the operator was still typing credentials, submitting them
- * skipped `final-review` entirely and jumped straight to `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.
  *
- * The real invariant the guard wants is: has the operator already been through
- * the final-review screen and finalized once? That's what `finalReviewPassed`
- * captures — it flips true in `onFinalize` on the GenerateReviewStep, not on
- * mere prefetch completion.
- *
- * Retained (and required) is `generateSessionId != null`, so we don't jump to
- * push-decision-gate before generation is even complete.
+ * 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;
+    return opts.cliNoCache;
 }
 /**
  * Skip-credentials spec — Task 2. When the operator pressed `s` on the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentful/experience-design-system-cli",
-  "version": "2.12.2-dev-build-4dc32df.0",
+  "version": "2.12.2-dev-build-e72ea10.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-4dc32df.0"
+    "@contentful/experience-design-system-types": "2.12.2-dev-build-e72ea10.0"
   },
   "devDependencies": {
     "@tsconfig/node24": "^24.0.3",