Skip to content

fix: recover docs when the deployed marker is unreadable - #11

Merged
zackees merged 1 commit into
gh-pagesfrom
fix/docs-manifest-recovery
Oct 22, 2026
Merged

zackees merged 1 commit into
gh-pagesfrom
fix/docs-manifest-recovery

Conversation

@zackees

@zackees zackees commented Oct 22, 2026

Copy link
Copy Markdown
Member

Fix

Treat an unreadable deployed publication marker as needing repair from the verified upstream manifest. Keep upstream validation strict and retain archive integrity checks before deployment.

Addresses CodeRabbit comment 4344932201 on #10. A corrupt or unavailable current marker should not block ordinary website-push recovery.

Verification

  • Updated real HTTP regression failed before the fix, then passed.
  • Invalid deployed metadata now requests an update; invalid upstream metadata still fails.
  • All 11 tests, Ruff and whitespace checks pass; primary review clean.

Follow-up to #10; contributes to #9.

@coderabbitai

coderabbitai Bot commented Oct 22, 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 49 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: a40ac2d4-b8da-4939-87b6-b742804bc861
📥 Commits

Reviewing files that changed from the base of the PR and between 41bf372 and a061580.

📒 Files selected for processing: 2
  • scripts/import_docs.py
  • scripts/test_import_docs.py
  • Autofix · 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.

@zackees
zackees merged commit 6f24625 into gh-pages Oct 22, 2026
2 checks passed
@zackees
zackees deleted the fix/docs-manifest-recovery branch October 22, 2026 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

1 participant