Emit create note actions#19
Conversation
Assisted-by: Codex:gpt-5.5
|
@codex review |
|
Codex Review: Didn't find any major issues. Another round soon, please! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
✅ Action performedReview finished.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthrough
ChangesUserCreateNote action emission
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related issues
Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Implements the Phase 1
Create(Note)core flow forfeder-core.When
Input::UserCreateNotetargets the configured local actor, core now:NoteCreate<Note>activity and records that activityAction::StoreObjectfor the created noteAction::SendActivity(Activity::CreateNote(...))once per known delivery targetDesign Notes
The core still performs no HTTP, storage backend writes, signing, delivery, async work, clock reads, or ID generation.
FederStatekeeps in-memory test/core state, whileAction::StoreObjecttells a future runtime that the created object should be persisted outside the portable core.Delivery is limited to known delivery targets. Shared inbox grouping, follower resolution, public addressing (
to/cc), mentions, media, and delivery workers remain out of scope for this issue.Validation
cargo fmt --checkcargo test -p feder-coremise run checkRefs #9
Summary by CodeRabbit