Skip to content

Add notification-to-chat prefill flow#541

Merged
ucguy4u merged 1 commit into
mainfrom
codex/issue-281-notification-chat-prefill
Jun 29, 2026
Merged

Add notification-to-chat prefill flow#541
ucguy4u merged 1 commit into
mainfrom
codex/issue-281-notification-chat-prefill

Conversation

@ucguy4u

@ucguy4u ucguy4u commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR introduces the missing in-app notification deep-link slice for Agent Skills.
Goal: let users jump from a scheduled reminder card back into Mind chat with the related reminder context prefilled.

Core outcome:

  • added an Open in chat action on reminder cards
  • wired the Mind chat route to accept a prefilled draft without auto-sending
  • added widget coverage for notification navigation and composer prefilling

Changes

Code

  • Added:
    • notification-to-chat prefill helper and notification screen action
    • widget test covering notification card navigation into chat
  • Updated:
    • ChatScreen to accept an optional initial draft
    • /mind/chat route to pass the prefill query parameter into chat
    • existing chat metadata test to verify draft prefilling

Logic

  • reminder cards build a prompt from title, message, and schedule context
  • chat opens with the composer pre-populated and cursor placed at the end
  • empty or missing reminder context still falls back safely to a normal chat route

API Changes (if any)

  • None.

Database Changes (if any)

  • None.

Observability / Logging

  • None.

Performance Impact

  • Latency: negligible
  • Throughput: none
  • Memory/CPU: negligible

Risks

  • prompt wording for reminder context is heuristic and may need later UX refinement
  • this only covers the in-app notification management path, not OS notification tap payloads
  • Rollback plan:
    • revert this PR

Testing

  • Unit tests:
    • flutter test test/features/agent_chat/presentation/screens/chat_screen_metadata_test.dart test/features/agent_chat/presentation/screens/notifications_screen_test.dart
  • Integration tests:
    • none
  • Manual testing:
    • not run

Deployment Notes

  • Config changes:
    • none
  • Order of deployment:
    • normal app rollout

Related Commits

  • fix(agent-chat): prefill chat from reminder cards

Notes

- add an Open in chat action for scheduled reminder cards
- pass reminder context into Mind chat through the route prefill contract
- cover composer prefill and notification navigation with widget tests

Delivers the in-app notification deep-link slice for issue #281 while keeping the broader skill-pack issue open for remaining work.
@github-actions

Copy link
Copy Markdown
Contributor

Plugin Module Size Gate

Policy: modules over 3 MB must be delivered as plugins; plugin modules over 5 MB must document cache management.

Module Size Type Status
No changed Dart/Flutter modules detected n/a n/a OK

@ucguy4u ucguy4u merged commit 5fe5e91 into main Jun 29, 2026
11 of 12 checks passed
@ucguy4u ucguy4u deleted the codex/issue-281-notification-chat-prefill branch June 29, 2026 16:13
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ❌ failure Title format, docs, bundled model guardrail
Code Quality ✅ success Analyze, formatting
Core Tests ✅ success Core package unit tests

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

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.

Add built-in Agent Skills pack and notification management

1 participant