Skip to content

Surface active pull request status - #185

Merged
ymichael merged 6 commits into
mainfrom
bb/inspect-thread-work-thr_yav8x4nauy
Jun 17, 2026
Merged

Surface active pull request status#185
ymichael merged 6 commits into
mainfrom
bb/inspect-thread-work-thr_yav8x4nauy

Conversation

@ymichael

Copy link
Copy Markdown
Collaborator

Summary

  • expand the existing GitHub PR read path to include checks, review state, mergeability, branch names, and updated time
  • normalize those raw GitHub facts into app-facing PR status, attention, checks, review, and mergeability summaries
  • surface the active thread PR summary in the Info panel and prompt context banner using the existing icon wrapper

Validation

  • pnpm exec turbo run test --filter=@bb/host-workspace --filter=@bb/server --filter=@bb/host-daemon --filter=@bb/host-daemon-contract --filter=@bb/server-contract --force
  • pnpm exec turbo run test --filter=@bb/app --force
  • pnpm exec turbo run typecheck --filter=@bb/domain --filter=@bb/host-workspace --filter=@bb/host-daemon-contract --filter=@bb/server-contract --filter=@bb/server --filter=@bb/app --force
  • git diff --check

Notes

  • active-thread only; no sidebar PR cache, webhooks, or review-comment handling in this slice
  • uses the existing Hugeicons-backed Icon wrapper; no lucide added

@ymichael
ymichael force-pushed the bb/inspect-thread-work-thr_yav8x4nauy branch from f5129b2 to 2c5ec93 Compare June 17, 2026 03:17
@ymichael
ymichael force-pushed the bb/inspect-thread-work-thr_yav8x4nauy branch from 2c5ec93 to 0fdd5af Compare June 17, 2026 18:24
@ymichael
ymichael marked this pull request as ready for review June 17, 2026 19:00
@ymichael
ymichael merged commit fa6e36e into main Jun 17, 2026
6 checks passed
@ymichael
ymichael deleted the bb/inspect-thread-work-thr_yav8x4nauy branch June 17, 2026 19:22
SawyerHood added a commit that referenced this pull request Jun 17, 2026
…read-cancel #199)

Resolve conflicts and reconcile our automations migration with main's
hand-authored thread-search migrations:

- Backfill drizzle snapshots for main's hand-authored 0039_thread_search and
  0040_thread_search_rowid_fts (replayed via drizzle-kit against main's schema,
  preserving main's FTS SQL); regenerate our automations migration as
  0041_add_automations with its own snapshot. Full snapshot chain now intact.
- migration-journal.test.ts: add a CI guard that every journal entry has a
  snapshot and the prevId chain is unbroken (prevents snapshot-less
  hand-authored migrations from recurring).
- migrate.test.ts: keep both cleanup paths in the thread-provenance rewind;
  add resetMigrationsAfterThreadSearch so the thread-search replay tests stay
  valid now that a migration (automations) lands after thread search.
- AppSidebar.tsx / ProjectList.tsx: keep both main's thread-search UI and our
  Automations nav entry.
- CLI timeline fixtures: add systemMessageKind/systemMessageSubject (required
  on the user conversation row since #192).
- pnpm install for main's new remark-breaks dependency.

Verified: turbo build typecheck lint test --force --continue -> 69/69 tasks pass.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
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