Conversation
Signed-off-by: David Viejo <dviejo@kungfusoftware.es>
Signed-off-by: David Viejo <dviejo@kungfusoftware.es>
Signed-off-by: David Viejo <dviejo@kungfusoftware.es>
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.
Successful deployments currently provide no route-level evidence of broken links or SEO regressions. This adds Site Crawl: a native TypeScript plugin with an administrator-only Temps sidebar UI, persistent reports, and automatic crawling after successful deployments.
Production-only automation is enabled by default once an administrator grants
events_read. Administrators can include other environments, exclude projects, pause automation, clear queued work, and cancel active crawls. Reports identify the project, environment, and deployment that triggered them.Behavior
Verification
bun run check: passed.bun test src: 44 passed, 146 assertions.bun run build: passed.bun audit: no vulnerabilities across 170 packages.Remaining limits
This has not been installed and triggered by a real deployment through a full Temps instance. Deployment integration evidence uses the compiled plugin with a local host protocol harness; successful automatic crawl content uses HTTP mocks. The separate browser crawl used a real public URL. No AI provider calls are used.
This PR does not publish a registry entry or release binary. The self-contained plugin folder can be published through the normal reviewed installation flow. Older hosts without permission discovery support manual crawls only. Crawls inspect server HTML, not JavaScript-rendered pages, and inspect the live URL at crawl time rather than an immutable deployment snapshot. Private/authenticated sites and external link checking are outside this version.
Follow-up verification
Tokenizer migration
Replaced Cheerio for HTML and sitemap parsing with bounded htmlparser2 callbacks. Added seven focused regression tests for metadata/entities, ignored subtrees, robots/nofollow, malformed HTML, field/link caps, sitemap namespaces, and excessive nesting. HTTP buffering stays bounded to 8 MiB; parsing does not construct a DOM. Real CLI documentation returned HTTP 200, 1,343,812 bytes, the correct title and 129 unique links. Final security review PASS after adding HTML/XML depth limits. Full suite: 44 passed, 146 assertions; native build/typecheck/audit passed.