style(admin/sqs): drop final test docstring PR attributions

Catches the three locations the prior round (7c63a7b) missed:
adapter/sqs_admin_test.go (TestAdminQueueSummary_CreatedAtUsesMillisNotHLC,
TestMetaAttributesForAdmin_IncludesQueueArnAndLastModified) and
main_admin_test.go (TestTranslateAdminQueuesError_LeaderChurn). Same
pattern: the why-rationale stays, only the "fifth-round Claude
review on PR #670" / "seventh-round Claude review on PR #670" /
"Codex P2 + Claude P1 on PR #670" attributions are dropped.

Pre-existing PR #634 references at main_admin_test.go:373 / :397
were added by a prior merged PR and are out of scope here.

Author: bootjp

adapter/sqs_admin_test.go

@@ -8,13 +8,13 @@ import (
 
 const testQueueArn = "arn:aws:sqs:us-east-1:000000000000:orders"
 
-// TestAdminQueueSummary_CreatedAtUsesMillisNotHLC pins the regression
-// from the fifth-round Claude review on PR #670: the admin
-// AdminDescribeQueue path was producing CreatedAt from
-// hlcToTime(meta.CreatedAtHLC), but sqsQueueMeta documents HLC as
-// "unsuitable for wall-clock display" and the SigV4 path
-// (sqs_catalog.go:942) reads CreatedAtMillis. Two failure modes the
-// test pins:
+// TestAdminQueueSummary_CreatedAtUsesMillisNotHLC pins the
+// invariant that the admin AdminDescribeQueue path derives
+// CreatedAt from sqsQueueMeta.CreatedAtMillis (the canonical
+// wall-clock field), not from hlcToTime(CreatedAtHLC) — the meta
+// struct documents HLC as "unsuitable for wall-clock display" and
+// the SigV4 path (sqs_catalog.go:942) reads CreatedAtMillis. Two
+// failure modes the test pins:
 //
 //  1. CreatedAtMillis == 0 must yield a zero time.Time so the JSON
 //     encoder's omitempty drops the field and the SPA renders "—"
@@ -58,14 +58,13 @@ func TestAdminQueueSummary_CreatedAtUsesMillisNotHLC(t *testing.T) {
 	})
 }
 
-// TestMetaAttributesForAdmin_IncludesQueueArnAndLastModified pins the
-// gap from the seventh-round Claude review on PR #670: the prior
-// metaAttributesForAdmin docstring claimed parity with
-// queueMetaToAttributes("All") but neither QueueArn nor
-// LastModifiedTimestamp was emitted. QueueArn is the AWS-shaped
-// identifier the SPA shows for change-tracking; LastModifiedTimestamp
-// is updated on SetQueueAttributes and is the only handle operators
-// have on "when did somebody last touch this queue's config".
+// TestMetaAttributesForAdmin_IncludesQueueArnAndLastModified pins
+// the parity contract between metaAttributesForAdmin and
+// queueMetaToAttributes("All"): QueueArn (the AWS-shaped identifier
+// the SPA shows for change-tracking) and LastModifiedTimestamp
+// (updated on SetQueueAttributes — the only handle operators have
+// on "when did somebody last touch this queue's config") must both
+// be present.
 func TestMetaAttributesForAdmin_IncludesQueueArnAndLastModified(t *testing.T) {
 	t.Parallel()
 

main_admin_test.go

@@ -433,7 +433,6 @@ func TestTranslateAdminTablesError_UnrelatedErrorPassesThrough(t *testing.T) {
 // still drop leadership inside deleteQueueWithRetry's Dispatch; the
 // resulting ErrLeaderNotFound / ErrNotLeader / wrapped suffixes must
 // classify as 503 leader_unavailable, not the generic 500 fallthrough.
-// Codex P2 + Claude P1 on PR #670 confirmed the original gap.
 func TestTranslateAdminQueuesError_LeaderChurn(t *testing.T) {
 	cases := []struct {
 		name string