Skip to content

Fix repository errors and DeepMind capture - #1

Merged
patrickschiller merged 1 commit into
mainfrom
agent/fix-github-errors-deepmind-capture
Jul 31, 2026
Merged

patrickschiller merged 1 commit into
mainfrom
agent/fix-github-errors-deepmind-capture

Conversation

@patrickschiller

Copy link
Copy Markdown
Owner

Summary

  • verify the configured GitHub repository before HTML or PDF uploads and surface a clear error when it is missing or inaccessible
  • add a dedicated Google DeepMind blog DOM adapter that captures the article sections while excluding cover controls and related-post cards
  • record DeepMind captures as google-deepmind-dom
  • document the behavior in English and German
  • add synthetic regression coverage for both bugs

Root cause

GitHub Contents API responses use 404 both for missing archive paths and for an unavailable repository. The existing code treated the first index-path 404 as an empty archive, so repository availability was not established explicitly.

The DeepMind page contains its actual post in top-level main > section elements, while its first article element is a related-content card. The generic fallback selected that card and saved almost none of the post.

User and privacy impact

Users now see an explicit popup error when the configured repository does not exist or the token cannot access it. DeepMind blog posts retain their full article content and omit related-post cards.

No browser permissions or token scopes change. Uploads make one additional read request to the existing GitHub repository metadata endpoint before writing. No storage format or deletion behavior changes.

Checks

  • node --test tests/capture-utils.test.js
  • python3 -m unittest discover -s tests -p "test_*.py"
  • git diff --check
  • inspected the live DeepMind page DOM to confirm the article is split across top-level sections and related posts are a separate final section

Signed-off-by: Patrick Schiller <p@trickschiller.de>
@patrickschiller
patrickschiller marked this pull request as ready for review July 31, 2026 20:57
@patrickschiller
patrickschiller merged commit a8f9240 into main Jul 31, 2026
1 check passed
@patrickschiller
patrickschiller deleted the agent/fix-github-errors-deepmind-capture branch July 31, 2026 21:00
patrickschiller added a commit that referenced this pull request Jul 31, 2026
Signed-off-by: Patrick Schiller <p@trickschiller.de>
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