Repository navigation
Toolify post-merge: secure embeds + validated manual-only release (DRAFT / NO-GO) - #41
Draft
appdevelopsk wants to merge 42 commits into
Draft
appdevelopsk wants to merge 42 commits into
appdevelopsk wants to merge 42 commits into
Conversation
…ts/tools/ShareBar.tsx
…s/embed-html.test.ts
…th-integration.test.mjs
…e 22 regression suite
…nd disabled workflows
…nd PR40/PR38 status
…gger variants and CLI exits
… regression suite
…ed existing policy
…ed existing policy
… manual-deploy tests
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.
Status: DRAFT / NO-GO
PR #40 is merged; superseded PR #38 is closed without merge. PR #41 contains post-merge hardening. Do not merge or deploy based on isolated test results. GitHub Actions were not invoked.
Implemented
on: workflow_dispatch), keeping VPS lock and release-SHA checks. Current main retains its preexisting auto-deploy workflow until an authorized merge.scripts/check-postmerge-embed-release.shchecks its exact branch, fetches current origin/main, requires ancestry, and uses the strictscripts/verify-manual-deploy.mjsparser to reject all unreviewed workflow event syntax (includingon: [push], inline event lists/mappings, duplicate events). It then runs four dependency-free verifier regression test groups, six embed Node tests, npm install, smoke/integration, typecheck, Vitest, validation, SEO/i18n/index audits and Next build.Executed evidence (not simulated)
node --test verify-manual-deploy.test.mjs: 4/4 passed, including actual CLI accept (0) and reject (4).bash -n: passed. Actual CLI rejectson: [push]with exit 4.Release blockers
bash scripts/check-postmerge-embed-release.shonfix/toolify-postmerge-embed-safety-20261107, fix any failures, review the workflow trigger change and browser behavior, then approve PR41. Owner performs manual production deploy andSITE_URL=https://toolify365.com npm run audit:live-growth.Full evidence:
docs/PR40_POSTMERGE_FOLLOWUP_2026-11-07.md. No traffic, conversion or revenue uplift claimed.