Skip to content

Add request correlation and per-request database telemetry - #305

Merged
samgbafa merged 11 commits into
mainfrom
skgbafa/tc-959-n2-request-db-telemetry
Oct 17, 2026
Merged

samgbafa merged 11 commits into
mainfrom
skgbafa/tc-959-n2-request-db-telemetry

Conversation

@samgbafa

@samgbafa samgbafa commented Oct 16, 2026 •

Copy link
Copy Markdown
Contributor

Part of TC-959.

Changes

  • Validate, normalize and echo request UUIDs; emit typed request-completion events with scoped database timings.
  • Use keyed account/space pseudonyms without adding raw identifiers, SQL or parameters to request events.
  • Share one startup-derived core pseudonymizer with length-prefixed database hashing and literal synthetic fixtures for F6 reuse.
  • Require explicit first-party origins for correlation forwarding. Document key custody, restart stability and instrumentation coverage/exclusions.

Verification

Head: a9bed3209be9082a639d7d6fafcf6e2846070391, after a normal current-main merge.

No release, deployment, shipper/schema change or F6 instrumentation is included. Performance measurements are separate release-cut evidence.

@samgbafa
samgbafa marked this pull request as draft October 16, 2026 18:13
@samgbafa

Copy link
Copy Markdown
Contributor Author

Review record for TC-959:

  • Astra, exact head e82447ed7dfb46e524cfbd32eddc4271fd2249e6: no actionable findings on the reviewed N2 production call paths.
  • Opus, exact head df6a75c2b52b3ed482a0d2d9ed412bead63fdabc: helper-delta source clearance; reported findings resolved, nothing new.

These were source reviews. Current-main resolution review is recorded separately at its exact head; release/deployment decisions remain separate.

@samgbafa

Copy link
Copy Markdown
Contributor Author

APPROVE: see TC-959 (reviewed exact head a9bed32)

https://linear.app/tinycloud-labs/issue/TC-959

@samgbafa
samgbafa marked this pull request as ready for review October 17, 2026 09:22
@samgbafa
samgbafa merged commit 96253a9 into main Oct 17, 2026
20 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