Alphea App

0.8-3 review findings 먹음¶

commit:server2|a3cb45c

첫 컷 (a0ca19e) 후 리뷰에서 나온 네 건 (Medium 3 + Low 1) 모두 같은 슬라이스에 접어 넣음.

1) Medium — bootstrap resume 가 server-validated. 첫 컷은 local expiry 만 확인 → stale/revoked/cross-principal session_id 를 프로세스 전체 동안 stamp 가능 (--auto-renew 꺼져있으면 영구). startup bootstrap 이 hard requirement 이니 resume 도 엄격 한와. Fix: tryResumeAgentSession 가 AgentSessionService.GetInfo 호출 (a) row 존재 (b) current bearer principal 소유 (cross-principal 는 server 가 PermissionDenied) (c) status="active". 다른 모든 케이스 (missing/revoked/expired/transport error) → fresh Create

2) Medium — handler input validation. 두 경계 leak. (a) metadata JSONB invalid JSON 이 CodeInternal 로 떨어짐 → 이제 handler 에서 json.Unmarshal probe → CodeInvalidArgument. (b) namespace lowercase 만 되고 scheme+address shape 검증 안 함 → normaliseAgentSessionNamespace 헬퍼 추가, wallet|token + lowercase 0x + 40 hex 규칙 (audit extractor 와 동일). 둘 다 empty 면 "no hint" valid shape 로 유지

3) Medium — audit GetAgentSession 이 sync 이어서 response path 차지 (200ms latency 추가). "session bookkeeping never blocks RPC" 약속과 모순. Fix: GetAgentSession + ClassifyAgentSession 도 off-request goroutine 으로 이동. session_status 가 INSERT 와 비동기 계산되지만 Go memory model 이 구조체 ref 의 happens-before 보증. chain integration test 가 여전히 fake writer 의 channel 에서 entry 받을 때 status 채워져있는지 검증하므로 계약은 observable

4) Low — DB-backed lifecycle tests. 二 신규 파일:

• internal/store/agent_sessions_test.go (TEST_DATABASE_URL):
  • TestAgentSessionLifecycle — create → get → touch → renew → revoke → revoked-renew error → revoke 멱등
  • TestAgentSessionMissingErrors — Get/Renew/Revoke 의 각 missing 케이스가 ErrAgentSessionNotFound 를 surfacing
  • TestClassifyAgentSession — nil/revoked/expired/active 4 sub-case (pure function, DB 불필요)
• internal/handler/agent_test.go (DB-free, nil DB 사용):
  • Unauthenticated / empty surface / invalid metadata JSON / invalid namespace_type / invalid namespace_id 모두 InvalidArgument
  • TestNormaliseAgentSessionNamespace 헬퍼 validator table (소문자/대문자, scheme rejection, short id, non-hex, missing 0x)

검증¶

go test  ./sdk ./internal/dshcore ./internal/server \
         ./internal/store ./internal/handler \
         ./cmd/dshai ./cmd/dsh ./cmd/sdktest      — all clean
go vet   ./...                                    — clean
go build ./...                                    — clean

DB-backed store tests (TEST_DATABASE_URL 의존) 는 환경이 없을 때 skip; dev Postgres (alphea_test DB) 에서는 실제 동작 검증 가능.

decision-log 의 2026-05-20 follow-up entry 에 네 건의 "왜 그 첨 구현에서 이 올가미가 나왔는지" 명시.

0.8-3 second-round review Lows 먹음¶

commit:server2|16036df

이전 follow-up (a3cb45c) 의 2차 리뷰에서 나온 두 Low 모두 같은 슬라이스에 접어 넣음.

1) Low — cross-principal / FailedPrecondition / NotFound 커버리지가 handler test 에 있어야. store.Renew/Revoke/GetAgentSession 은 principal-blind (session_id 만 권수); principal 비교는 handler 에서 일어남. 이전 cut 이 cross-principal guard test 를 store 테스트에 놓아 regression 신호가 머될 수 없었다. Fix: DB-backed handler test 6종 신규 (TEST_DATABASE_URL 의존):

• TestAgentSessionCreateFreezesPrincipal — bearer 에서 파생됨 검증
• TestAgentSessionRenewRejectsCrossPrincipal / RevokeRejectsCrossPrincipal / GetInfoRejectsCrossPrincipal
• TestAgentSessionRenewOnRevokedMapsToFailedPrecondition
• TestAgentSessionMissingMapsToNotFound (Get/Renew/Revoke 3 sub-case)

node_test.go 의 openTestDB(t) 재사용. 파일 주석도 DB-free vs DB-backed 분할 정확하게 재작성.

2) Low — metadata validator helper 분리. 이전 cut 의 TestAgentSessionCreateAcceptsEmptyMetadata 가 nil-DB panic 을 defer recover() 로 삼키던 구조 — 테스트가 validator 의 올/그름과 무관하게 세이브. Fix: validateAgentSessionMetadata([]byte) error 를 handler body 에서 추출. handler 는 그저 호출 후 에러를 CodeInvalidArgument 로 lift. TestValidateAgentSessionMetadata 가 이 helper 를 직접 10개 케이스로 (nil, empty slice, valid object/array/primitive/null, plain text, truncated, unbalanced) 검증 — DB X, recover X.

부수 fix (dev-DB run 출처): TestAgentSessionLifecycle 가 req.Metadata vs fetched.Metadata byte-for-byte 비교 했는데 Postgres JSONB 가 round-trip 시 공백 정규화 → {"model":"test"} → {"model": "test"}. 이제 둘 다 unmarshal 후 reflect.DeepEqual — 계약은 "same JSON document" 이지 "same byte sequence" 가 아님.

검증 (두 구성)¶

# TEST_DATABASE_URL unset:
go test  ./sdk ./internal/dshcore ./internal/server \
         ./internal/store ./internal/handler \
         ./cmd/dshai ./cmd/dsh ./cmd/sdktest      — all clean (DB-backed skip)
go vet   ./...                                    — clean
go build ./...                                    — clean

# TEST_DATABASE_URL=postgres://…/alphea_test:
v21/v22 migrations 자동 적용. 6개 DB-backed handler test +
lifecycle/missing store test 모두 PASS.

decision-log 에 2026-05-20 follow-up entry 추가 — 두 finding 의 root cause 명시.