Skip to content

feat(snooze): choose custom wake date or duration - #830

Draft
rynfar wants to merge 1 commit into
pylonfrom
feat/custom-snooze-dates-20260924
Draft

rynfar wants to merge 1 commit into
pylonfrom
feat/custom-snooze-dates-20260924

Conversation

@rynfar

@rynfar rynfar commented Sep 24, 2026 •

Copy link
Copy Markdown
Collaborator

Change

Adapts upstream 3be02ae5791480b8c8bf3a13020a932378eca1ba (#11800) to Pylon's existing snooze command, capability gate, receipt, Undo notice, and mobile thread-row actions. Users can choose a local date/time or an elapsed duration in the web/desktop thread menu or mobile v2 row. The five existing presets remain. A shared resolver rejects past/invalid dates and nonexistent spring-forward times; repeated fall-back wall times use the first occurrence, and duration days mean 24 elapsed hours.

The web dialog uses native browser date/time inputs, avoiding a new calendar dependency. The browser/OS supplies their locale-specific calendar presentation. Upstream e1cbb705273a261d26b489f35a7ddca6567d9663 specifically configures an upstream custom calendar's week start and is not claimed as adopted here. The web request host keys each dialog to its own request so a stale close or submit cannot resolve a replacement request; unmount cancels only its matching request.

Scope and verification

  • 112 focused shared resolver, menu, and mounted web dialog tests, including DST and request replacement/unmount; client-runtime/web/mobile scoped typechecks and lint pass; formatting and diff checks pass.
  • Independent review PASS on exact commit ef3375339e653ccd461d5590bca41f1ec8aeba38, including the request-ownership correction.
  • No live browser or native device interaction was performed. Native picker appearance and browser date-input locale presentation need interactive acceptance.

Owning review: #736. No ledger cursor advancement or merge is included.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@vercel

vercel Bot commented Sep 24, 2026

Copy link
Copy Markdown

Deployment failed for project pylon-marketing with the following error:

Resource is limited - try again in 24 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/pylon-code?upgradeToPro=build-rate-limit

@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 14.0 KiB 13.9 KiB −73 B (−0.5%) 15.1 KiB ✅
Codex Thread snapshot wire 7.2 KiB 7.2 KiB −3 B (−0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 6.8 KiB 6.7 KiB −70 B (−1.0%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 58.0 KiB 58.0 KiB −88 B (−0.1%) 66.4 KiB ✅
Codex Live turn messages 10 8 −2 (−20.0%) 21 ✅
Claude Total thread wire 14.0 KiB 14.0 KiB −3 B (−0.0%) 15.1 KiB ✅
Claude Thread snapshot wire 7.2 KiB 7.2 KiB −13 B (−0.2%) 7.3 KiB ✅
Claude Live turn WebSocket wire 6.7 KiB 6.8 KiB +10 B (+0.1%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 58.9 KiB 58.9 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 10 10 0 (0.0%) 21 ✅

Baseline: 7ed3f71 · PR result: ef33753 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 115.6 KiB
  • Claude decoded thread snapshot: 116.3 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL 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