trent_larson/crowd-funder-for-time-pwa
4
1
Fork 2
Code Issues Pull Requests 22 Releases 5 Wiki Activity

wip_new_notifications #196

Open
anomalist wants to merge 7 commits from wip_new_notifications into master
pull from: wip_new_notifications
merge into: trent_larson:master
Conversation 0 Commits 7 Files Changed 33 +1234 -201
anomalist commented 2025-09-05 13:09:41 +00:00
Owner
Copy Link

Build Architecture Guard PR Template

Change Level

  • Level: L1 / L2 / L3 (pick one)

Why: …

Scope & Impact

  • Files & platforms touched: …
  • Risk triggers (env / script flow / packaging / SW+WASM /
    Docker / signing): …
  • Mitigations/validation done: …

Commands Run (paste exact logs/snips)

  • Web: npm run build:web / :prod
  • Electron: npm run build:electron:dev / package step
  • Mobile: npm run build:android:test / iOS equivalent
  • Clean/auto-run impacted scripts

Artifacts

  • Names + sha256 of artifacts/installers:

Artifacts:

<name-1>  <sha256-1>
<name-2>  <sha256-2>

Docs

  • BUILDING.md updated (sections): …
  • Troubleshooting updated (if applicable)

Rollback

  • Verified steps (1–3 cmds) to restore previous behavior

L3 only

  • ADR link:

ADR: https://…

# Build Architecture Guard PR Template ## Change Level - [ ] Level: **L1** / **L2** / **L3** (pick one) **Why:** … ## Scope & Impact - [ ] Files & platforms touched: … - [ ] Risk triggers (env / script flow / packaging / SW+WASM / Docker / signing): … - [ ] Mitigations/validation done: … ## Commands Run (paste exact logs/snips) - [ ] Web: `npm run build:web` / `:prod` - [ ] Electron: `npm run build:electron:dev` / package step - [ ] Mobile: `npm run build:android:test` / iOS equivalent - [ ] Clean/auto-run impacted scripts ## Artifacts - [ ] Names + **sha256** of artifacts/installers: Artifacts: ```text <name-1> <sha256-1> <name-2> <sha256-2> ``` ## Docs - [ ] **BUILDING.md** updated (sections): … - [ ] Troubleshooting updated (if applicable) ## Rollback - [ ] Verified steps (1–3 cmds) to restore previous behavior ## L3 only - [ ] ADR link: ADR: https://…
anomalist added 4 commits 2025-09-05 13:09:41 +00:00
chore: create new notification plan 5110c17fba
chore: still tweaking plan a427a9e66f
refactor(docs): split notification system docs into plan and implementation cfeb920493 
Replace monolithic notification-system-implementation-plan.md with focused
strategic plan (notification-system-plan.md) and detailed implementation
guide (notification-system-implementation.md). Both documents now perfectly
aligned with TimeSafari codebase patterns including:

- Actual Settings type extension pattern (JSON strings for complex objects)
- Real useNotifications composable stub signatures with eslint-disable
- Verified logger exports and safeStringify usage
- Confirmed PlatformServiceMixin.$saveSettings integration
- Validated migration system registerMigration patterns

Documents are production-ready with accurate code examples verified
against actual TimeSafari infrastructure.
chore: update plan and rulesets 74e4a20ef9
anomalist added 1 commit 2025-09-07 10:50:48 +00:00
docs: apply documentation references model agents directive to notification system docs 79b226e7d2 
- Remove duplicate sync checklists from notification-system-plan.md
- Fix markdown table formatting (remove double pipes) in executive summary
- Streamline cross-document references and eliminate redundant content
- Consolidate canonical ownership statements across all three docs
- Improve document structure and readability per directive guidelines

Files modified:
- doc/notification-system-executive-summary.md
- doc/notification-system-implementation.md
- doc/notification-system-plan.md

Compliance: Follows @docs/documentation_references_model_agents.mdc directive
for eliminating redundancy, centralizing context, and optimizing reference placement.
anomalist added 1 commit 2025-09-07 13:15:26 +00:00
docs: merge notification system docs into single Native-First guide 0dcb1d029e 
- Consolidate 5 notification-system-* files into doc/notification-system.md
- Add web-push cleanup guide and Start-on-Login glossary entry
- Configure markdownlint for consistent formatting
- Remove web-push references, focus on native OS scheduling

Reduces maintenance overhead while preserving all essential information
in a single, well-formatted reference document.
anomalist added 1 commit 2025-09-08 04:22:35 +00:00
docs: update notification system for shared SQLite + clear T–lead 464a825a7b 
- Add SQLite Ownership & Concurrency section with WAL mode details
- Add DB Path & Adapter Configuration with shared DB setup
- Clarify T–lead governs prefetch attempts, not arming
- Add TTL-at-fire check callout for stale notification prevention
- Add DB Sharing acceptance criteria (visibility, WAL overlap, version safety)
- Update GLOSSARY.md with shared DB, WAL, and user_version definitions
- Remove plugin-owned DB references in favor of single shared database

Files modified:
- doc/notification-system.md
- doc/GLOSSARY.md

Compliance: Implements shared SQLite architecture with clear T–lead prefetch
semantics and WAL-based concurrency for app/plugin coordination.
This pull request has changes conflicting with the target branch.
  • .husky/pre-commit
  • .husky/pre-push
  • BUILDING.md
  • README.md
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin wip_new_notifications:wip_new_notifications
git checkout wip_new_notifications
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
anomalist jose jsnbuchanan kentbull trentlarson
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: trent_larson/crowd-funder-for-time-pwa#196
Delete Branch "wip_new_notifications"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?