Skip to content

fix(automation): rebuild highlights after final bridge restore - #110

Merged
Sam-24-dev merged 1 commit into
mainfrom
fix/local-reddit-runner-integrity
Sep 8, 2026
Merged

Sam-24-dev merged 1 commit into
mainfrom
fix/local-reddit-runner-integrity

Conversation

@Sam-24-dev

Copy link
Copy Markdown
Owner

Summary

  • version the local Sunday Reddit runner used by TechnologyTrend-RedditBaselineWeekly
  • rebuild only home_highlights.json from the final restored bridge set before integrity validation
  • replace broad cleanup with an exact-file snapshot/rollback for tracked and ignored run outputs

Root cause

sync_assets.py built Home highlights from stale datos/latest GitHub data. The runner then restored canonical non-Reddit bridges but did not rebuild Home highlights, so the integrity gate correctly rejected the mixed artifact set.

Safety

  • publication steps remain after all integrity and date checks
  • any prepublication failure restores prior bytes and removes only exact files created by that run
  • no git clean command is used
  • canonical bridges and timestamps are not altered to force a pass
  • no ETL, scheduled task, workflow dispatch, or deploy was executed

Validation

  • python -m pytest tests/test_local_reddit_runner.py -q — 3 passed
  • python -m pytest tests -q — 233 passed, 1 existing Pandera warning
  • python -m compileall -q backend tests
  • ruff check tests/test_local_reddit_runner.py
  • bandit -q -ll tests/test_local_reddit_runner.py
  • PowerShell parser validation for both automation files
  • detect-secrets-hook --baseline .secrets.baseline on the staged diff
  • git diff --cached --check

Operational note

After merge, the separate automation clone must receive the merged runner before the next scheduled execution. This PR does not alter or run Task Scheduler.

Copilot AI lite review requested due to automatic review settings September 8, 2026 02:29

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Sam-24-dev
Sam-24-dev merged commit dead49b into main Sep 8, 2026
3 checks passed
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