Skip to content

Record the Credentials file for the Resend key (ADR 0007) - #195

Merged
JacobStephens2 merged 2 commits into
mainfrom
email-setup
Sep 30, 2026
Merged

JacobStephens2 merged 2 commits into
mainfrom
email-setup

Conversation

@JacobStephens2

Copy link
Copy Markdown
Owner

Summary

Docs only: the domain language and decision record for #190, landed ahead of its tickets #191 and #192 so their agents find them on main.

 CONTEXT.md
+  **Credentials**: per-machine secrets file, apart from the User config;
+                   a secret in the environment wins over it
   **Setup**: writes the User config
+             … and the Credentials when a key is entered
 docs/adr/
   0005-run-notifications-through-resend.md
-    key only ever comes from the environment
+    key never comes from the User config; ADR 0007 adds the Credentials fallback
+  0007-resend-key-in-a-credentials-file.md   # why: cron reads no shell profile

Evidence

Merge Danger

Door: two-way

Markdown only; no code or tests touched.

Blast Radius: none

🤖 Generated with Claude Code

https://claude.ai/code/session_01GyLEwAarEKDeDouyMka9xr

JacobStephens2 and others added 2 commits September 30, 2026 18:17
Add the Credentials term to CONTEXT.md and amend Setup to write it.
ADR 0007 records why the Resend API key can live in a file thirdshift
reads itself: cron and non-interactive shells never see a shell-profile
export. ADR 0005's env-only consequence now points to it.

Docs for #190, ahead of #191 and #192.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GyLEwAarEKDeDouyMka9xr
Resolve the CONTEXT.md conflict on Credentials by keeping main's entry,
written with #191's implementation and review; keep this branch's Setup
amendment and ADRs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GyLEwAarEKDeDouyMka9xr
@JacobStephens2
JacobStephens2 merged commit 2a8487e into main Sep 30, 2026
5 checks passed
@JacobStephens2
JacobStephens2 deleted the email-setup branch September 30, 2026 18:21
@JacobStephens2 JacobStephens2 mentioned this pull request Sep 30, 2026
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.

1 participant