Skip to content

Preserve explicit model choices during catalog hydration - #1887

Open
matthewpenkala wants to merge 1 commit into
storytold:mainfrom
matthewpenkala:contrib/preserve-explicit-model-choice
Open

Preserve explicit model choices during catalog hydration#1887
matthewpenkala wants to merge 1 commit into
storytold:mainfrom
matthewpenkala:contrib/preserve-explicit-model-choice

Conversation

@matthewpenkala

Copy link
Copy Markdown

Summary

  • Track whether a model selection is automatic or explicitly chosen by the user.
  • Let automatic selections follow the available default after catalog hydration.
  • Preserve a still-valid explicit selection instead of silently replacing it.
  • Rebind an explicit same-ID choice to the rebuilt catalog model so refreshed capabilities are not lost.

Why

Catalog hydration rebuilds model instances and may change the available default. Without selection provenance, the reconciliation step could either overwrite a user's explicit choice or retain a stale model object with outdated capabilities.

Validation

  • Focused selector-store suite: 7/7 passed.
  • Automatic fallback, explicit preservation, same-ID capability refresh, and page-scoped defaults are covered.
  • git diff --check and contribution-scope scans pass.

Origin and scope

This change was isolated from a broader downstream integration and reduced to a generic, self-contained upstream contribution. Private deployment/release-specific material and artifacts are intentionally excluded.

@netlify

netlify Bot commented Aug 11, 2026

Copy link
Copy Markdown

👷 Deploy request for artcraft-webapp pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 4267590

@netlify

netlify Bot commented Aug 11, 2026

Copy link
Copy Markdown

Deploy Preview for storyteller-docs ready!

Name Link
🔨 Latest commit 4267590
🔍 Latest deploy log https://app.netlify.com/projects/storyteller-docs/deploys/6a7b93a350147700085112c7
😎 Deploy Preview https://deploy-preview-1887--storyteller-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Aug 11, 2026

Copy link
Copy Markdown

Deploy Preview for artcraft-dashboard ready!

Name Link
🔨 Latest commit 4267590
🔍 Latest deploy log https://app.netlify.com/projects/artcraft-dashboard/deploys/6a7b93a3e6ac570008560c82
😎 Deploy Preview https://deploy-preview-1887--artcraft-dashboard.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant