Skip to content

Phase 4: restore fork settings and web push sidecars - #35

Merged
claude-do merged 1 commit into
sync/upstream-20260426from
phase4/settings-notifications
Apr 26, 2026
Merged

claude-do merged 1 commit into
sync/upstream-20260426from
phase4/settings-notifications

Conversation

@claude-do

Copy link
Copy Markdown
Collaborator

Summary

  • restore the fork-only settings store and sidecar section in upstream settings
  • reapply Codex native-notification suppression via session-scoped app-server overrides
  • restore web push registration, delivery, persistence, and deep-link navigation on the fresh upstream base

Verification

  • bun fmt
  • bun lint
  • bun typecheck
  • bun run --cwd apps/server test --run src/notifications/policy.test.ts src/notifications/http.test.ts src/fork/notifications/intentResolver.test.ts src/provider/Layers/CodexAdapter.test.ts src/orchestration/Layers/ProviderCommandReactor.test.ts
  • bun run --cwd apps/web test:browser --run src/settings/ForkSettingsSection.browser.tsx src/components/settings/SettingsPanels.browser.tsx

Review

  • two parallel subagent code reviews completed with no remaining blockers in the implemented phase4 diff

@claude-do
claude-do merged commit ad383fe into sync/upstream-20260426 Apr 26, 2026
2 of 4 checks passed
@claude-do
claude-do deleted the phase4/settings-notifications branch April 26, 2026 14:07
@github-actions github-actions Bot added size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Apr 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XXL vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant