Automate widget installation and static verification#49
Closed
mariojgt wants to merge 11 commits into
Closed
Conversation
|
This PR is too large for detailed analysis—consider breaking it into smaller, focused changes. 🎯 oversized — strongly consider breaking this down 🛡️ Standards: no pre-flight fit check ran for this change — wire |
…into agent/automatic-widget-install
Rename the testConnect directory to test-build for clearer naming and consistency with project conventions.
Establish clear processes for maintaining the adversarially-tuned install prompt and setup guides. Introduce MAINTAINING.md as the primary reference for safe edits (with load-bearing vs. safe-to-edit categories and five-artifact map). Update RELEASING.md with the recommended Release workflow that auto-computes semver and dispatches Publish, plus manual recovery path. Clarify across docs that the fixture installs published code, so unpublished CLI/guide/README changes require plan B or release-first. Update CLI message to recommend the package-manager path. Fix test-build directory references and sync prompt-related metadata.
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
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.
What changed
scan, persist its issuing endpoint atomically, and install one managed rolling-CDN widget in every required global source shell.mark-build --strictstructurally stamp and verify complete static HTML, require explicit--static-outputfor SSR-capable stacks, and reject ambiguous populated output trees unless--diris supplied.testConnectVite fixture, generated/ignored source handling, field-test alignment, and expanded integration/unit coverage.Why
The connector needs the UUID before compilation so the widget is visible immediately after installation and the same identity is baked into static production output. Previous manual instructions could miss the build lifecycle, duplicate widgets/sites, or report success without sitewide coverage.
Validation
npm run typechecknpm test— 303 tests passed across 16 filesnpm run buildtestConnect: npm run fixture:reset-source && npm run build:staticgit diff --checkorigin/mainRelease gates
This is intentionally a draft. Do not publish the automatic embedding flow yet. The backend currently uses the client-visible UUID for manifest writes and unclaimed-site claiming, and bare provisioning still lacks a server-side idempotency key for timeout-after-commit recovery. The public widget identity, authenticated manifest-write credential, and scoped one-time claim credential must be separated and tested first. Coordinate the widget CDN release before the connector release, then deploy the matching website guides.