Skip to content

refactor(session): consolidate internal session modules - #628

Merged
benvinegar merged 4 commits into
mainfrom
refactor/architecture-phase-3-session
Jul 28, 2026
Merged

refactor(session): consolidate internal session modules#628
benvinegar merged 4 commits into
mainfrom
refactor/architecture-phase-3-session

Conversation

@benvinegar

Copy link
Copy Markdown
Member

Summary

  • consolidate Hunk-owned session code under src/session/ by role: shared contracts, daemon clients, agent CLI, mounted-app bridge, and broker composition
  • preserve the existing packages/session-broker-* reusable runtime boundaries
  • keep the shared daemon HTTP protocol and schemas at the session root, separate from agent-only CLI code

Validation

  • bun run typecheck
  • bun run lint
  • focused session/client/agent/broker tests (165 passed before the final client-only move; 53 focused tests after it)
  • bun run test:integration (64 passed; one known pre-existing atomic-save watch failure)

This PR description was generated by Pi using OpenAI GPT-5

@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Too many files changed for review. (57 files found, 50 file limit)

Bypass the limit by tagging @greptile-apps to review.

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hunk-web Ready Ready Preview Jul 28, 2026 4:47pm

Request Review

@benvinegar
benvinegar merged commit d4aeb9e into main Jul 28, 2026
12 checks passed
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