Skip to content

docs: reconcile public GitHub PR state - #214

Merged
YoneRai12 merged 1 commit into
mainfrom
codex/public-github-state-reconciliation
May 20, 2026
Merged

docs: reconcile public GitHub PR state#214
YoneRai12 merged 1 commit into
mainfrom
codex/public-github-state-reconciliation

Conversation

@YoneRai12

Copy link
Copy Markdown
Owner

Scope

Validation

  • git diff --check passed.
  • Changed-file secret scan: no secret values found; token hits are existing explanatory wording only.
  • Changed-file local path / username / hostname scan: no hits.
  • Changed-file hidden Unicode scan: no hits.
  • Changed-file mojibake scan: no four-question-mark sequence and no replacement characters.
  • src/cogs/ora.py: unchanged.
  • reference_clawdbot: untouched.

Non-claims

This does not claim production readiness, shipping completion, official cloud completion, hybrid completion, full security backlog completion, full dependency backlog completion, persistent memory, Google login, Discord gateway completion, Tools/MCP completion, provider ecosystem completion, or src/cogs/ora.py resolution.

Traceability

@YoneRai12
YoneRai12 merged commit 499692f into main May 20, 2026
2 checks passed
@YoneRai12
YoneRai12 deleted the codex/public-github-state-reconciliation branch May 20, 2026 17:13

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request introduces a new maintenance document for public GitHub state reconciliation and updates the existing triage documentation. The review identified potential issues with the gh pr list and gh api commands, noting that the current limit of 100 may lead to inaccurate pull request counts as the backlog grows. Increasing these limits to 1000 is recommended to ensure accuracy.

- Follow-up branch: `codex/public-github-state-reconciliation`
- Current baseline: public `main` after PR #203.
- `origin/main`: `f636c482031021b9d21aeea1cdef1f0252e51ece`
- GitHub source of truth: `gh pr list --state open --limit 100 --json number --jq length`

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The --limit 100 flag in the gh pr list command will cause an incorrect count if the number of open pull requests exceeds 100. Consider increasing this limit to a higher value (e.g., 1000) to ensure the count remains accurate as the backlog grows.

Suggested change
- GitHub source of truth: `gh pr list --state open --limit 100 --json number --jq length`
- gh pr list --state open --limit 1000 --json number --jq length


Use GitHub API / `gh` as the operational source of truth for PR state.

- `gh pr list --state open --limit 100 --json number --jq length`: `43`

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The --limit 100 flag in the gh pr list command will cause an incorrect count if the number of open pull requests exceeds 100. Consider increasing this limit to a higher value (e.g., 1000) to ensure the count remains accurate as the backlog grows.

Suggested change
- `gh pr list --state open --limit 100 --json number --jq length`: `43`
- gh pr list --state open --limit 1000 --json number --jq length: 43

Use GitHub API / `gh` as the operational source of truth for PR state.

- `gh pr list --state open --limit 100 --json number --jq length`: `43`
- `gh api "/repos/YoneRai12/YonerAI/pulls?state=open&per_page=100" --jq length`: `43`

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The gh api command using per_page=100 only returns the first page of results, which will lead to an incorrect count if there are more than 100 open pull requests. It is recommended to use gh pr list with a sufficiently high limit, as done in the previous line, for a more robust count.

Suggested change
- `gh api "/repos/YoneRai12/YonerAI/pulls?state=open&per_page=100" --jq length`: `43`
- gh pr list --state open --limit 1000 --json number --jq length: 43

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