Skip to content

feat(relay): improve relay progress UX#77

Merged
zikolach merged 19 commits into
mainfrom
docs/notify-session-compaction-progress
Jun 14, 2026
Merged

feat(relay): improve relay progress UX#77
zikolach merged 19 commits into
mainfrom
docs/notify-session-compaction-progress

Conversation

@zikolach

@zikolach zikolach commented Jun 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

Summary

  • add safe compaction progress notifications for paired messenger bindings
  • coalesce live progress updates and use Telegram edit-in-place with safe fallbacks
  • reduce normal-mode progress noise by keeping volatile model/tool bookkeeping verbose-only
  • archive completed OpenSpec changes and sync their spec deltas into main specs
  • propose follow-up OpenSpec changes for aggregated tool-call progress and remote /new session handoff

Validation

  • npm run typecheck
  • npm test
  • openspec validate --all --strict

Copilot AI review requested due to automatic review settings June 13, 2026 13:26
Copilot AI reviewed Jun 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces an OpenSpec change proposal to notify paired messenger bindings when a Pi session compaction starts and when it completes, with delivery controlled by each binding’s progress mode (suppressed only for quiet).

Changes:

  • Adds requirements specifying compaction start/completion notifications and safety/nonfatal delivery constraints.
  • Documents design decisions and trade-offs for integrating session_before_compact / session_compact into existing progress notification plumbing.
  • Adds an implementation/test/validation task checklist and initializes the change with .openspec.yaml.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
openspec/changes/notify-session-compaction-progress/proposal.md Explains motivation, scope, and expected impact of compaction progress notifications.
openspec/changes/notify-session-compaction-progress/design.md Details how to map Pi compaction hooks onto existing progress events and progress-mode policy.
openspec/changes/notify-session-compaction-progress/specs/messenger-relay-sessions/spec.md Specifies normative requirements and scenarios for delivery, suppression, safety, and nonfatal behavior.
openspec/changes/notify-session-compaction-progress/tasks.md Lists implementation and test tasks needed to land the change.
openspec/changes/notify-session-compaction-progress/.openspec.yaml Registers the change as spec-driven with creation date metadata.

Comment thread openspec/changes/notify-session-compaction-progress/design.md Outdated
Copilot AI review requested due to automatic review settings June 13, 2026 14:10
Copilot AI reviewed Jun 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.

Comment thread extensions/relay/adapters/slack/runtime.ts Outdated
Comment thread extensions/relay/adapters/discord/runtime.ts Outdated
Copilot AI review requested due to automatic review settings June 13, 2026 14:29
Copilot AI reviewed Jun 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 18 out of 18 changed files in this pull request and generated 1 comment.

Comment thread extensions/relay/runtime/extension-runtime.ts
Copilot AI review requested due to automatic review settings June 13, 2026 19:27
Copilot AI reviewed Jun 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 27 out of 27 changed files in this pull request and generated 2 comments.

Comment thread extensions/relay/notifications/progress.ts
Comment thread extensions/relay/adapters/discord/runtime.ts Outdated
@zikolach zikolach requested a review from Copilot June 13, 2026 19:33
Copilot AI reviewed Jun 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 27 out of 27 changed files in this pull request and generated 1 comment.

Comment thread extensions/relay/notifications/progress.ts
@zikolach zikolach requested a review from Copilot June 13, 2026 19:39
Copilot AI reviewed Jun 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 27 out of 27 changed files in this pull request and generated 3 comments.

Comment thread extensions/relay/adapters/telegram/adapter.ts
Comment thread extensions/relay/notifications/progress.ts Outdated
Comment thread openspec/changes/archive/2026-06-14-coalesce-live-progress-updates/tasks.md
@zikolach zikolach changed the title docs(openspec): propose compaction progress notifications feat(relay): improve progress notifications Jun 13, 2026
Copilot AI reviewed Jun 13, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 33 out of 33 changed files in this pull request and generated 2 comments.

Comment thread extensions/relay/adapters/telegram/runtime.ts Outdated
Comment thread extensions/relay/broker/process.js
Copilot AI review requested due to automatic review settings June 14, 2026 10:32
Copilot AI reviewed Jun 14, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 46 out of 63 changed files in this pull request and generated 2 comments.

Comment thread extensions/relay/notifications/progress.ts
Comment thread openspec/changes/remote-new-session-handoff/proposal.md
@zikolach zikolach changed the title feat(relay): improve progress notifications feat(relay): improve relay progress UX Jun 14, 2026
@zikolach zikolach requested a review from Copilot June 14, 2026 11:05
Copilot AI reviewed Jun 14, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 46 out of 63 changed files in this pull request and generated 1 comment.

Comment thread extensions/relay/runtime/extension-runtime.ts
@zikolach zikolach requested a review from Copilot June 14, 2026 11:14
Copilot AI reviewed Jun 14, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 46 out of 63 changed files in this pull request and generated 2 comments.

Comment thread extensions/relay/adapters/telegram/runtime.ts
Comment thread extensions/relay/broker/process.js
@zikolach zikolach requested a review from Copilot June 14, 2026 11:30
Copilot AI reviewed Jun 14, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 46 out of 63 changed files in this pull request and generated 1 comment.

Comment thread extensions/relay/broker/process.js
@zikolach zikolach requested a review from Copilot June 14, 2026 12:01
Copilot AI reviewed Jun 14, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 46 out of 63 changed files in this pull request and generated 2 comments.

Comment thread extensions/relay/adapters/telegram/runtime.ts Outdated
Comment thread extensions/relay/broker/process.js
@zikolach zikolach requested a review from Copilot June 14, 2026 12:17
Copilot AI reviewed Jun 14, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 46 out of 63 changed files in this pull request and generated no new comments.

@zikolach zikolach merged commit b13e936 into main Jun 14, 2026
2 checks passed
@zikolach zikolach deleted the docs/notify-session-compaction-progress branch June 14, 2026 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@zikolach