Skip to content

docs(lane1): reverify outbound sender-trust item — DKIM fixed on main (PR #141), postal address blocked on operator fact - #179

Merged
nish3451 merged 1 commit into
mainfrom
lane1-outbound-sender-trust-reverify-20260815-105038
Aug 15, 2026
Merged

docs(lane1): reverify outbound sender-trust item — DKIM fixed on main (PR #141), postal address blocked on operator fact#179
nish3451 merged 1 commit into
mainfrom
lane1-outbound-sender-trust-reverify-20260815-105038

Conversation

@nish3451

Copy link
Copy Markdown
Collaborator

Lane 1 third re-verification report for the outbound sender-trust item.

Verification (live, 2026-08-15)

  • dkimSelector: "resend" is set on main (PR fix(sales): bind outbound sender trust to Resend DKIM selector #141, fcf5ff3) and the live resend._domainkey.tinystudio.io record resolves.
  • npm run send:setup:advisory (live DNS): SPF found, DMARC found, DKIM found.
  • npm run send:setup (strict) exits 1 with exactly one warning: missing physical postal addresssenderPhysicalAddress is "" in growth-brain/ops/agency-config.json.
  • node scripts/test-operator-check-strictness.mjs passes (exit 0).

Why no product code change

The DKIM half is already merged and verified live. The postal-address half is a NEEDS-NISH operator fact: CAN-SPAM requires the real published business location (or PO box / private mailbox), which cannot be invented by code. The configure script already refuses placeholders and requires a real-looking address, so no code gap remains.

Unblock

  1. npm run send:configure -- --physical-address="<real address>" --dkim-selector=resend
  2. Re-run npm run send:setup — should pass with SPF, DMARC, DKIM, and the address verified.

No product files changed; this PR contains only the lane report.

… (PR #141), postal address blocked on operator fact (#177)

Lane 1 third re-verification report: dkimSelector=resend is set on main
(PR #141, fcf5ff3) and the live resend._domainkey.tinystudio.io record
resolves; SPF, DMARC, and DKIM all verify. The only remaining sender
warning is the empty senderPhysicalAddress, which requires a real
published business address (CAN-SPAM) that no code change can supply.
No product files changed.

Co-authored-by: CommandCodeBot <noreply@commandcode.ai>
@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 15, 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: 31 minutes

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: 9d7ae1b2-2964-4614-816c-1ebaeb7ad962

📥 Commits

Reviewing files that changed from the base of the PR and between 949c0c7 and c662c02.

📒 Files selected for processing (1)
  • .lane/reports/lane1-outbound-sender-trust-reverify-20260815-105038.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 29a8707 into main Aug 15, 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