Repository navigation
fix(skills): port remaining support-assistant fixes after 0.2.0 - #23
Open
hitesh-shetty-cstk wants to merge 1 commit into
Open
hitesh-shetty-cstk wants to merge 1 commit into
hitesh-shetty-cstk wants to merge 1 commit into
Conversation
…erge These were made on the original branch after it was folded into the 0.2.0 release, so they did not reach it: - Open in New Tab is scoped to Live Preview; Visual Editor does not support it and always renders in its canvas iframe. - Missing Visual Experience navigation checks the stack's Live Preview setting before treating it as provisioning. - The CORS entry no longer rules out a Contentstack-side cause, and its verification says curl does not enforce CORS. - GraphQL preview requests keep access_token and Content-Type. - livePreviewQuery: live_preview is the only required key and an empty one resets the instance; both key spellings are accepted. - The editButton exclude config applies only where init() runs. - Cache guidance separates app and CDN caches from SDK caching and says the hash rotates on a timer rather than on edits. - The framework examples say which stackSdk to pass per Delivery SDK. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up to #21. Eight fixes to
cms-live-preview-visual-editor-support-assistantlanded on the original branch (#18) after it was folded into the 0.2.0 release, so they never reached it. This ports them onto the merged text and skips everything #21 already covered (hash presence checks, Management Token scope, Onboarding Check before "what changed", SSR evidence from server logs).Changes
faq-integration.mdandcontact-support.md. Visual Editor does not support it and always renders the site in its canvas iframe, so recommending it for an SSO-gated Visual Editor site sends users down a dead end.visual-experience-navigation-missing-for-the-stack.curldoes not enforce CORS.access_tokenandContent-Typeon every request; the preview headers are added, not swapped in.livePreviewQuery:live_previewis the only required key, an empty one resets the instance to published content, and bothcontent_type_uidandcontentTypeUidspellings are accepted (checked againstcontentstack-typescriptsrc/stack/stack.ts).excludeconfig is applied only byinit(), so builds that import the SDK without callinginit()need the purge flag or a preview-only import.contentstackv3 SDK forcesNETWORK_ELSE_CACHEonce a real hash is set, and the hash rotates on a 24-hour timer, not on edits.stackSdk: the framework examples note thatstack.configis for@contentstack/delivery-sdkand the legacy SDK takes theStackinstance.Test plan
node scripts/validate.mjspasses (23 skills, 7 manifests)next🤖 Generated with Claude Code