Skip to content

Refresh canonical static-site gate dependency - #3128

Closed
ktsaou wants to merge 2 commits into
masterfrom
deps/refresh-canonical-build-gate
Closed

ktsaou wants to merge 2 commits into
masterfrom
deps/refresh-canonical-build-gate

Conversation

@ktsaou

@ktsaou ktsaou commented Oct 8, 2026 •

Copy link
Copy Markdown
Member

The canonical static-site validator still installed source-map-js 1.2.1. Consume the published owner bundle from netdata-labs/seo @ dd8d11f0, updating that dependency to 1.2.2 and its manifest checksum. The complete export is byte-identical to its owner.

Gate code, ruleset 10, Node 22, baselines, renderer inputs and indexing policy are unchanged. This is the Learn handoff under SEO SOW-0038; Website receives the same bundle independently.

Validation:

  • Clean Node 22.23.2 gate install and export verification passed.
  • Fresh owner installation passed all 53 existing tests; owner repository validation passed in 113.6 seconds.
  • The updated validator passed the previously qualified 2,032-page production corpus with zero regressions or accepted findings. The renderer is unchanged; this local check does not claim a new renderer build.
  • Independent exact dependency, manifest and export review passed.

Merge after the already-published owner commit. No generated content or unrelated dependency updates are included.


Summary by cubic

Consumes the canonical static-site validator's published owner bundle, updating the source-map-js dependency from 1.2.1 to 1.2.2 and its manifest checksum. The consumer gate assertion is aligned with the owner bundle's export, which is byte-identical to its owner; gate code, ruleset 10, Node 22, baselines, renderer inputs, and indexing policy are unchanged.

Written for commit 2826a91. Summary will update on new commits.

View guided diff Turn on auto-fix

@netlify

netlify Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for netdata-docusaurus ready!

Name Link
🔨 Latest commit 2826a91
🔍 Latest deploy log https://app.netlify.com/projects/netdata-docusaurus/deploys/6ac82a5ab9eb0500087238c7
😎 Deploy Preview https://deploy-preview-3128--netdata-docusaurus.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 10 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b1619a72-3c07-4bdd-b727-d71d02718bf5
📥 Commits

Reviewing files that changed from the base of the PR and between 3e846ee and 2826a91.

⛔ Files ignored due to path filters (1)
  • scripts/site-build-gate/package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (2)
  • scripts/site-build-gate/manifest.json
  • tests/site_build_gate.test.js
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

View guided diff | Re-trigger cubic

@ktsaou
ktsaou marked this pull request as draft October 9, 2026 09:36
@ktsaou

ktsaou commented Oct 9, 2026

Copy link
Copy Markdown
Member Author

Superseded by #3136 (#3136), which combines all supported Learn package, runtime, tooling and copied-browser-asset updates with a fresh inventory and combined validation. This proposal's history and branch are retained; review the combined draft instead.

@ktsaou ktsaou closed this Oct 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant