Skip to content

docs(lane1): reverify outbound sender-trust item — DKIM fixed on main (PR #141), postal address still NEEDS-NISH - #192

Merged
nish3451 merged 1 commit into
mainfrom
lane1-outbound-sender-trust-reverify-20260817
Aug 17, 2026
Merged

docs(lane1): reverify outbound sender-trust item — DKIM fixed on main (PR #141), postal address still NEEDS-NISH#192
nish3451 merged 1 commit into
mainfrom
lane1-outbound-sender-trust-reverify-20260817

Conversation

@nish3451

Copy link
Copy Markdown
Collaborator

Lane 1 re-verification: outbound email blocked on sender trust

Lane: tinystudio-in lane 1 (worktree tinystudio-in-lane1-20260817-110531)
Date: 2026-08-17
Branch: lane1-outbound-sender-trust-reverify-20260817

Outcome

Re-verification of this item confirms the DKIM half is fixed and live on main (PR #141, dkimSelector: "resend" resolves in DNS; SPF, DMARC, DKIM all found). The single remaining sender-trust blocker is the empty senderPhysicalAddress in growth-brain/ops/agency-config.json, which is a real operator fact — a physical postal address must be the operator's actual published business location and cannot be invented by code. No product code change is possible or needed; the remaining half is a NEEDS-NISH operator action.

The one item

[unreviewed-by-opus] Outbound email stays blocked on sender trust: empty physical postal address and empty dkimSel

DKIM half — already fixed on main and verified live

  • growth-brain/ops/agency-config.json at origin/main carries dkimSelector: "resend" (set by PR fix(sales): bind outbound sender trust to Resend DKIM selector #141, fcf5ff3b, merged 2026-08-14).
  • npm run send:setup:advisory (live DNS, 2026-08-17) reports SPF found, DMARC found, DKIM found at resend._domainkey.tinystudio.io.
  • dkimCandidates: [] (no discovery needed once the selector is configured).

Postal address half — blocked on a real operator fact (NEEDS-NISH)

  • senderPhysicalAddress is "" in growth-brain/ops/agency-config.json at origin/main.
  • npm run send:setup (strict) exits 1 with exactly one warning: missing physical postal address.
  • The CAN-SPAM physical-address requirement is an operator fact: it must be the real published business location (or PO box / private mailbox). It cannot be defaulted, auto-detected, or invented. scripts/configure-sender-setup.mjs already refuses placeholders and requires a real-looking address, so no code gap remains on this side.

What unblocks the remaining half

  1. Nish publishes a real business postal address (or PO box / private mailbox) and saves it:
    npm run send:configure -- --physical-address="<real address>" --dkim-selector=resend
  2. Re-run npm run send:setup — it should then pass with SPF, DMARC, DKIM, and the address all verified.

Files touched

  • /home/nish/workspaces/agent-state/lanes/tinystudio-in/lane-1.jsonclaims list only (this report); no other field changed.
  • .lane/reports/lane1-outbound-sender-trust-reverify-20260817.md — this report (unique to this lane).
  • No product code or config files changed.

See .lane/reports/lane1-outbound-sender-trust-reverify-20260817.md for the full report. This is a docs-only re-verification PR.

… (PR #141), postal address still NEEDS-NISH (#lane1)
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Your trial has ended. Reactivate Greptile to resume code reviews.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@nish3451, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 27 minutes

Limit details: You’ve used all 1 included review currently available under your plan.

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: d5852b79-80a8-45e3-a321-951e9dc2d894

📥 Commits

Reviewing files that changed from the base of the PR and between a6cd49b and e2d1f91.

📒 Files selected for processing (1)
  • .lane/reports/lane1-outbound-sender-trust-reverify-20260817.md

Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands.

@nish3451
nish3451 merged commit 53ea7a9 into main Aug 17, 2026
2 checks passed
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