Repository navigation
Owner-requested branch cleanup and master protection #265
Description
Owner request and remaining scope
The owner requested removal of unnecessary branches and protection of master on 2026-09-30, followed by repository cleanup on 2026-10-18. This is scoped maintenance, not a blocker for analyzer/Reader delivery. Do not change the hourly scheduler.
Current verified state — 2026-10-18
- Authenticated connector login
andy-zen-dev; repository reportspush=true,admin=false,maintain=false. - Master has active ruleset
24273338(Protect master): PR required, deletion and non-fast-forward updates prohibited, zero mandatory approvals. No administration/protection mutation was made in this cleanup. - The observed ruleset does not require named CI checks or resolved review threads. Those requirements and automatic merged-branch deletion remain separate owner/admin settings.
- SCRIP-CORPUS-087: recover Petersburg extraction-strategy preflight #264/chore: finalize SCRIP-CORPUS-087 state #266 were already merged. SCRIP-SITE-007: readable offline full-work diagnostic reports #268/SCRIP-SITE-008: canonical full-work report and reader-first homepage #272 are now merged; current verified master is
9412ee54ab1a56c7c81f6ff4d290e39d83f49630. - New Reader 0.2: repair release blockers, consolidate ready issues and roadmap #276 reuses
scrip-site-two-full-work-galleryand preserves its unique module/tests plus the Reader integration; no new branch or branch deletion in this cleanup. SCRIP-METRIC-003: private local analysis with JSON and offline HTML #270/Reader 0.2: integrated literary analyzer, installable CLI and five-route site #274 are retained as superseded drafts pending successor acceptance.
The September body observations protected=false / empty rulesets are historical, not current blockers. Their original execution receipts remain in this issue's discussion.
Safe completion criteria
- Preserve master/default/protected refs, all open PR heads/bases and every active dependency or unique/unproven change.
- Before supported authorized deletion, produce an exact branch/head allow-list with fresh content/merge preservation evidence. Squash/rebase merges require content/supersession proof rather than ancestry alone. Re-read current PR references, protection and SHA, use expected-SHA deletion leases, and abort on drift or denial.
- Preserve the earlier flagged Bulgakov/Klim branches until their unique content is reconciled against landed code; do not resurrect or delete stale versions blindly.
- Record actual deleted/retained refs and before/after count. Never simulate deletion by updating refs, force-push shared branches, or bypass a rejected cleanup operation through another endpoint/workflow/account.
- A future required-check rule needs an always-reporting verification check; do not make a path-filtered workflow globally mandatory and strand unrelated PRs. No self-approval requirement for the sole working account.
Scoped blockers and reopening triggers
MISSING_TOOL: the discovered connector actions expose no branch-deletion or administration-setting mutation. Permission boundary: current identity has no repository admin permission. No new cleanup/deletion write was attempted here, and no new safety denial is inferred from successful interactive source writes.
Resume those mutations only when a legitimate supported capability and its necessary owner authorization/permissions exist. Keep this issue open until the remaining deletion/protection criteria are actually verified; independent product work continues through #276 and the compact queue.
Activity
andy-zen-dev commentedon Sep 30, 2026
Execution receipt for the owner's cleanup/protection request:
andy-zen-dev; repository permissionspush=true,admin=false,maintain=false.master@27f54858d20000244bb8e80342038d80ea510b45was observedprotected=false; repository rulesets list was empty.create_treewas blocked by OpenAI safety checks. No tree/commit/branch/workflow execution or branch deletion is claimed. Do not retry the same blocked mutation via another write endpoint.--apply, validates owner/admin capability for applying both operations, preserves default/protected/open-PR/unique branches, proves integration through master/merged-PR ancestry, and deletes with exact-SHA leases. It creates no remote helper branch or workflow. It also enables and reads backdelete_branch_on_mergeafter protection is verified.Required verificationworkflow has been prepared but is NOT committed, run, or required.Remaining blocker: execute the reviewed maintenance under a legitimate owner/admin GitHub session, or restore a supported authorized mutation path; then record exact before/after inventory and verify active master protection. Issue #265 remains open. No scheduler changes.
andy-zen-dev commentedon Oct 6, 2026
Fresh interactive owner-requested triage, 2026-10-06; this supersedes only the stale current-state observations above:
masteris now covered by active ruleset24273338(Protect master): pull request required, deletion and non-fast-forward updates prohibited, zero mandatory approving reviews, no current-user bypass. No protection/admin mutation was made in this session.abe7bb889ef98437ba9faef5a79b023499556991; do not select either for another merge.scriptorium-corpus-086-bulgakov-revision-freezebranch pending recovery review, not blind merge/deletion. Exact head2e2144782b88238fb6206343b56c22d18e47bbd2, filetests/test_bulgakov_revisions.py, blobbf08511c93ed9d4033565709d94ec35374aa3f8econtains a literal backslash-n between statements on line 17. Reconstructed bytes match that Git blob;ast.parsefails withSyntaxError: unexpected character after line continuation character. Recovery must also compare against the already landed Bulgakov implementation/tests rather than replacing them with this stale branch.scrip-corpus-020-klim-literary-body@0da222f824b98f1526cd8702469bab669051c376is 261 commits behind and 20 ahead of current master by ancestry. These counts are not content-preservation evidence; compare the five changed paths and accepted later implementations before proposing adoption or deletion.