Skip to content

Preserve recovered collector and work-filter batch - #1

Open
sedge-bot wants to merge 4 commits into
fielding:mainfrom
sedge-bot:sedge/recovered-collector-work-filter
Open

Preserve recovered collector and work-filter batch#1
sedge-bot wants to merge 4 commits into
fielding:mainfrom
sedge-bot:sedge/recovered-collector-work-filter

Conversation

@sedge-bot

Copy link
Copy Markdown

Summary

  • Preserve the recovered Yabai organize collector and Cursor dump helpers
  • Add work-filter/config support to aggregation and CLI output paths
  • Update sample config and cost test expectations for the recovered batch

Verification

  • npm test
  • npm run build
  • git diff --check HEAD^..HEAD

@sedge-bot

Copy link
Copy Markdown
Author

Heartbeat re-verification at head 6cbe233 (2026-05-15): PR is OPEN/MERGEABLE/CLEAN with no configured GitHub checks. Local verification passed in /opt/sedge/.openclaw/workspace/clones/thinktax-validation-2026-05-05: npm test (37 passed), npm run build, and git diff --check github/main..HEAD. Clone remained clean after verification.

@sedge-bot

Copy link
Copy Markdown
Author

Heartbeat re-verification for PR #1 at 6cbe233:

  • GitHub state: OPEN / MERGEABLE / CLEAN; no configured status checks surfaced.
  • npm test passed: 5 files, 37 tests.
  • npm run build passed via tsup.
  • git diff --check github/main..HEAD passed.
  • Clone is clean after verification (git status --short --untracked-files=all returned no output).

No duplicate PR or patch-only handoff created.

@sedge-bot

Copy link
Copy Markdown
Author

Heartbeat update: added a small billing-trust follow-up on top of this PR at 8c49dfe. Reprocess now preserves billing provenance alongside the mode (session_registry/high vs config_default/default) for Claude/Codex/OpenClaw events instead of flattening everything into a bare billing enum.\n\nVerification passed locally: npm test (39 tests), npm run build, and git diff --check.

@sedge-bot

Copy link
Copy Markdown
Author

Heartbeat update: pushed 11d009d to this PR to make default/lower-confidence billing visible in summaries. Aggregates now expose default_billing_count and lower_confidence_billing_count, and text totals show ~ plus [N default-billed] when totals rely on config-default billing assumptions.\n\nVerification: failing-first aggregation regression, npm test (40 tests), npm run build, git diff --check HEAD~1..HEAD.

@sedge-bot

Copy link
Copy Markdown
Author

Heartbeat update at head ddd8c55: added the next billing-honesty slice so mixed/unknown billing windows are counted separately, included in lower-confidence totals, and shown in human-readable status/popup lines as [mixed-billing]/[unknown-billing] instead of looking like subscription certainty.\n\nVerification: observed failing-first Vitest coverage for mixed/unknown aggregation and CLI formatting, then passed npm test (42 tests), npm run build, and git diff --check. GitHub currently reports OPEN with mergeability/status UNKNOWN and no status checks surfaced.

@sedge-bot

Copy link
Copy Markdown
Author

Follow-up after GitHub refreshed PR metadata: PR #1 at ddd8c55 now reports OPEN/MERGEABLE/CLEAN with no status checks configured. Local verification remains: npm test (42 tests), npm run build, git diff --check.

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