Skip to content

fix: make daily content maintenance honest and repository-aware - #6

Merged
ECorreia45 merged 2 commits into
mainfrom
fix/daily-content-maintenance-2026-10-17
Oct 17, 2026
Merged

ECorreia45 merged 2 commits into
mainfrom
fix/daily-content-maintenance-2026-10-17

Conversation

@ECorreia45

Copy link
Copy Markdown
Contributor

Summary

  • Preserve both daily cron schedules: factory triage at 09:23 UTC and maintenance at 10:11 UTC
  • Replace root-npm assumptions with repository-specific checks; real failures fail the run
  • Keep useful deduplicated failing-PR triage; remove automatic merging and explicitly report unsupported automatic source/dependency repair
  • Restrict unattended and issue-triggered AI proposals to README.md documentation drafts, with read-only example context and no claims that code issues were implemented
  • Reuse fail-closed provider/publication guards; zero-exit quota errors fail, protected files/Git state stay unchanged, and checks rerun before publishing a draft
  • Do not expose provider credentials or run AI calls in pull-request validation; do not persist checkout credentials or give package checks GitHub/OpenAI token environment variables

Repository-specific validation

Parse all 34 root extensionless JavaScript snippets, including names with spaces. Validate README structure/local links and JSON/workflow syntax. Add focused deep-clone regression coverage; this is not a complete snippet behavior suite.

Source repair

The existing deep clone snippet was truncated. Restore its return and closing function delimiter. Regressions cover nested objects, arrays, Maps, Sets and primitive values.

Verification

  • 40 maintenance/adapter tests passed on Python 3.12
  • Bash syntax and actionlint 1.7.7 passed for both workflows
  • Full repository validation passed locally on Node 22.23.3 (and Node 24), including applicable regressions and the actual nested tutorial build
  • No live AI/provider billing calls, merges, credential changes, or billing changes were made
  • Remote CI for this exact commit must be checked separately

Limits and rollout

This draft does not activate changes on the default branch. Human review and merge are required. The existing provider credit problem remains an external blocker; this change exposes it instead of masking it as successful maintenance. Automated source repair remains explicitly unsupported until adequate test coverage exists. Draft publication guards are not an execution sandbox.

@ECorreia45
ECorreia45 marked this pull request as ready for review October 17, 2026 22:50
@ECorreia45
ECorreia45 merged commit a6c32d3 into main Oct 17, 2026
2 checks passed
@ECorreia45
ECorreia45 deleted the fix/daily-content-maintenance-2026-10-17 branch October 25, 2026 16:59
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