Skip to content

fix(file-service): restore share notifications - #253

Open
devin-ai-integration[bot] wants to merge 3 commits into
mainfrom
devin/1787626572-share-notification-sns-fix
Open

devin-ai-integration[bot] wants to merge 3 commits into
mainfrom
devin/1787626572-share-notification-sns-fix

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Aug 25, 2026

Copy link
Copy Markdown

Summary

Symptom / blast radius: file_shared notification publishes were routed to a nonexistent SNS topic ending in -v2, so shares returned an SNS error and recipients did not receive notifications.

Root cause (config/env): the file-service image baked FILE_SHARE_EVENT_ALWAYS_FAIL=true; EventPublisher::file_shared therefore selected SNS_TOPIC_ARN-v2 instead of the configured SNS topic.

Fix: remove the enabled image default so deployed file-service instances use the existing off-by-default application configuration and publish shares to SNS_TOPIC_ARN. Add a regression guard covering modern and legacy ENV syntax, truthy/quoted values, multi-assignment declarations, and backslash continuations. Upload failure and seeded-content settings are unchanged.

Verification:

  • cargo test: 26 passed
  • cargo fmt --check
  • cargo clippy --all-targets -- -D warnings
  • Mutation check: a backslash-continued truthy Dockerfile assignment makes the regression guard fail
  • Built image contains no FILE_SHARE_EVENT_ALWAYS_FAIL
  • API and browser shares of Annual Planning 2027.txt return 201 and publish file_shared to otterworks-events; zero -v2 or publish-failure log entries
  • Recipient sees the shared file and in-app notification
  • Recording: https://app.devin.ai/attachments/77335a43-cb4a-4c93-be4a-bc80c16fa257/pr253-share-edited.mp4

Successful share toast

Follow-up observed: after the successful share, the owner's "People with access" list did not refresh, although recipient access and notification delivery were confirmed. This appears separate from the SNS outage and is not changed here.

Related independent work, left unchanged: #251, #248, #245.

Link to Devin session: https://app.devin.ai/sessions/b9ea2445f63348559338e25689db482f


Devin Review

Status Commit
⚪ Not started

Run Devin Review

Devin Review (Staging)
Open in Devin Review

Co-Authored-By: bot_apk <apk@cognition.ai>
@devin-ai-integration
devin-ai-integration Bot requested a review from a team August 25, 2026 03:01
@devin-ai-integration

Copy link
Copy Markdown
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

devin-ai-integration[bot]

This comment was marked as resolved.

Co-Authored-By: bot_apk <apk@cognition.ai>
devin-ai-integration[bot]

This comment was marked as resolved.

Co-Authored-By: bot_apk <apk@cognition.ai>

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 0 new potential issues.

Open in Devin Review

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.

0 participants