Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

fix(cc): reconcile stale quick-roster rows - #267

#267
Merged
github-actions[bot] merged 1 commit into
mainanipotts/claude-code-tips:mainfrom
codex/pro/cc-quick-live-roster-2026-07-24anipotts/claude-code-tips:codex/pro/cc-quick-live-roster-2026-07-24Copy head branch name to clipboard
Jul 25, 2026
Merged

fix(cc): reconcile stale quick-roster rows#267
github-actions[bot] merged 1 commit into
mainanipotts/claude-code-tips:mainfrom
codex/pro/cc-quick-live-roster-2026-07-24anipotts/claude-code-tips:codex/pro/cc-quick-live-roster-2026-07-24Copy head branch name to clipboard

Conversation

@anipotts

Copy link
Copy Markdown
Owner

problem

cc-quick roster treated every row with ended_at_ms IS NULL as live. Dirty exits left 1,607 historical rows open on ap-pro, so the Codex bash path reported years of stale tasks as active even though the MCP path already had a five-minute freshness contract.

change

  • apply the same configurable five-minute freshness window to roster, target resolution, and digest joins
  • mark stale rows ended at their last observed timestamp without deleting history
  • reject malformed freshness configuration
  • add fixture-backed regression coverage

verification

  • bash -n plugins/cc/bin/cc-quick
  • bun test tests/: 45 pass
  • bunx tsc --noEmit

Mirror the MCP server freshness window in cc-quick, mark dirty-exit rows ended without deleting history, and exclude stale peers from sends and digest joins.

Verified with bash syntax, 45 Bun tests, and TypeScript noEmit.
@github-actions

Copy link
Copy Markdown
Contributor

quality gate: all checks passed

  • PR description -- present
  • version stamps -- no doc files changed
  • hook conventions -- no shell scripts changed

automated check -- see CLAUDE.md for conventions.

@github-actions
github-actions Bot merged commit f20a022 into main Jul 25, 2026
10 of 13 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

@claude @codex please review this pr per .github/AI_REVIEW_RUBRIC.md.

lanes:

  • claude: prose, voice, api/contract shape, skill + hook + plugin surface correctness, tests for the public path.
  • codex: bugs, security, type/shell correctness, dependency hygiene, ci config.

output one pr comment each in the exact shape from the rubric. if an app is not installed on this repo, the mention sits unanswered and ci stays green.

@chatgpt-codex-connector

Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.