docs(todo): add a standing human/maintainer action queue#1035
Merged
Conversation
Creates _project/TODO/main/active/human-action-required.yaml — a single running index of actions only the human maintainer can perform (GitHub repo-admin changes, Code-Owner reviews, human recruitment), which agents in this project cannot do (the sandbox git remote rejects branch/default-branch changes, admin APIs are out of scope, CODEOWNERS review must be human). Seeded with the current queue: - w1: Code-Owner review + merge PR #1027 (in-repo main->release rename). - w2 (needs w1): GitHub-side branch rename main->release + ruleset recreate. - w3: set default_branch=develop (+ verify scheduled workflows) — also activates the §2.8 workflow_run companion and the release-canary cron. - w4: recruit one external contributor for the Phase-2 submission dry-run. - w5 (optional): delete the 5 dead orphan branches (agent couldn't; harmless — they're allowlisted for the orphaned-commit detector). Each work unit cross-references its authoritative source TODO and gives the exact command. New maintainer/human actions get appended here so there is one place to see what's waiting on the maintainer. todo validate + check-graph green. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JFdZAbavgrKaNLe4YZa3zf
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 22bbb7a96a
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
17 tasks
joeharris76
added a commit
that referenced
this pull request
Jul 8, 2026
…1038) Sweep of chatgpt-codex-connector review comments that landed after their respective PRs merged (#1032, #1037, #1035, #1030, #1031, #1028, #1025, #1024, #1022, #1020, #1019). Each fix includes a regression test verified to fail before the fix and pass after: - qpc-05/#1032: thread the real DataFrame plan-capture-error cause through to both the .plans.json companion (existing_errors=) and the main result row (plan_capture_error field); reset per-run capture-error state on ExpressionFamilyAdapter (qpc-15 w1-w3 implemented). - #1037: fix qpc-13's scope_limit path (_spark_helpers.py lives directly under benchbox/platforms/, not benchbox/platforms/dataframe/) and add the missing deps.needs: [qpc-05] to qpc-13 and qpc-15 planning YAMLs. - #1035: gate human-action-required.yaml's branch-rename admin action (w2) on the default-branch switch (w3) too, matching the source TODO's A-before-B sequencing. - #1030: release-cut's git commit now passes --no-verify, since the curation step already deleted .pre-commit-config.yaml and _project/scripts/ (the local hook entrypoints) from the working tree. - #1031: fix `python -m benchbox platform list` -> `benchbox platforms list` (plural, actual console script) across both DuckLake TODO planning files. - #1028: correct plan_changed against the full tree-walk fallback so a pure fingerprint-encoding-version bump (v1->v2) is never misreported as a changed plan; thread fingerprint_version through PlanMetadata/PlanHistory so version-boundary crossings are excluded from flapping/version-history diffs. - #1025: normalize a "Z"-suffixed timestamp to "+00:00" before datetime.fromisoformat(), which only accepts "Z" from Python 3.11 - the project's floor is 3.10. - #1024: recursively strip physical_operator.platform_metadata across the full operator tree for anonymized exports, not just the top-level raw_explain_output - many parsers copy raw EXPLAIN text into per-node metadata under platform-specific key names. - #1022: match each repeated query failure to its own error entry (FIFO per query_id) instead of always reusing the first; synthesize a standalone FAILED row for any error not actually consumed by a queries[] row, not just for query_ids absent from queries[] entirely. - #1020: document orphaned-commit-detector.yml as a third instance of the known "scheduled workflow absent from default branch" class in repo-admin-settings.md; use the latest PR (by number) as the branch-diff cutoff and skip the branch when that PR closed without merging, so an abandoned closed PR's commits aren't misreported as orphans. - #1019: move the qpc-01 planning YAML from TODO/ to DONE/ (already Completed). Claude-Session: https://claude.ai/code/session_01SKw2UhgHuzYPo3y5MfarD6 Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Creates
_project/TODO/main/active/human-action-required.yaml— a single running index of the actions only the human maintainer can perform (GitHub repo‑admin/settings changes, Code‑Owner reviews, human recruitment). Agents in this project cannot do these (the sandbox git remote rejects branch/default‑branch changes, admin APIs are out of scope, CODEOWNERS review must be human), so this gives one scannable place to see "what's waiting on me." New such actions get appended here.Seeded queue
main→releaserename)main→release+ ruleset recreatePATCH default_branch=develop+ verify scheduled workflows — also activates the §2.8 companion + release‑canary cronEach unit cross‑references its authoritative source TODO and gives the exact command;
verification:entries check whether each admin action has been applied (gh pr view 1027,branches/release,.default_branch).Type of Change
Testing
todo_cli validate→ 1260/1260 valid;check-graph→ clean (108 items). New file only;_indexes/not committed (generated/gitignored).Notes
This is an index, not a re‑design — it deliberately does not duplicate each task's full spec, and it never performs the actions (agents update status/notes; the human executes). It's the standing home for future maintainer/human actions too.
🤖 Generated with Claude Code
https://claude.ai/code/session_01JFdZAbavgrKaNLe4YZa3zf
Generated by Claude Code