Repository navigation
Follow-up to #5: UTF-8 exhaustive test + Cloudflare Pages config checks - #6
Merged
Merged
Conversation
Every 1-, 2- and 3-byte tail after 'a' (leads 80-FF, all continuations plus non-continuation probes) is classified as text exactly when a fatal streaming TextDecoder says a valid completion exists, and src/ and landing/lib/ agree on all of them. No mismatches were found. docs/deploy.md lists the expected Pages settings (production branch main, auto production deploys on, output dir landing, no build command). test/deploy_config.test.js asserts what the repo can prove, and scripts/check-pages-settings.sh checks the dashboard read-only.
scripts/check-pages-settings.mjs replaces the shell script. It checks the repository, production branch, automatic production deployments, preview setting (all), build command, output and root directories, and the exact domain set, and exits 1 on any drift. It reads a saved API response with --from-file <path|->. test/check_pages_settings.test.js runs in npm test with a passing fixture, a negative case per check, and a check that the docs/deploy.md table equals EXPECTED.
Deploying patchling with
|
| Latest commit: |
44019d2
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://87ca4987.patchling.pages.dev |
| Branch Preview URL: | https://cursor-js5-followup-cloudfla.patchling.pages.dev |
--from-file and stdin now go through the same envelope check as live mode: anything but success:true with an object result (success:false, a missing success or result, result:null, invalid JSON) exits 2 even when the project data matches. pr_comments_enabled is checked and documented.
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
2 participants
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.
Addresses Ember's #5 points. UTF-8: exhaustive 1-3 byte tail test vs TextDecoder(fatal), 0 mismatches, code unchanged. Cloudflare: Pages settings verified read-only; adds docs/deploy.md, scripts/check-pages-settings.sh, CI repo-config test. npm test 184 pass / 0 fail / 4 live skipped.