Skip to content

feat: add nightly sitemap drift detection with Mattermost alert#100

Open
nicoalba wants to merge 3 commits into
mainfrom
sitemap-flow
Open

feat: add nightly sitemap drift detection with Mattermost alert#100
nicoalba wants to merge 3 commits into
mainfrom
sitemap-flow

Conversation

@nicoalba
Copy link
Copy Markdown
Contributor

feat: add nightly sitemap drift detection with Mattermost alert

@vercel
Copy link
Copy Markdown

vercel Bot commented May 21, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

2 Skipped Deployments
Project Deployment Actions Updated (UTC)
nf-theme-sandbox Ignored Ignored Preview May 21, 2026 8:30pm
unified-doc-preview Ignored Ignored Preview May 21, 2026 8:30pm

Request Review

Copy link
Copy Markdown
Member

@dovholuknf dovholuknf left a comment

Choose a reason for hiding this comment

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

wrong pr.. hold on... :)

nicoalba and others added 2 commits May 21, 2026 20:06
- Rewrite check-sitemap-drift.mjs: compare new build against cached
  baseline (not live prod), check redirect stubs for each removed path,
  exit 1 on unresolved removals so the publish is aborted
- Update publish-unified-doc.sh: pass baseline path and build dir,
  remove || true so drift failures actually block the script
- Update publish.yml: add actions/cache restore/save for baseline,
  drop the wrong notify-sitemap-drift job, add inline drift alert
  steps (if: failure()) that send to Mattermost doc-alerts when
  unresolved removals are found

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Expose has_drift as a job output and skip notify-mattermost when
drift already sent its own alert.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

2 participants