fix(selfserve): harden managed observe cli readiness

[michiosw]

Where We Are

The managed observe CLI stack is ready, but review found error-handling gaps around corrupt installation state, auth breadcrumb cleanup, and stream cursor typing.

Where We Want To Go

Keep the existing CLI stack PRs focused and add one cleanup PR that makes runtime diagnostics and persisted state safer before review.

How do we get there

This branch reports corrupt installation state as an error, returns and logs auth breadcrumb delete failures, preserves managed stream cursor JSON compatibility with stricter in-memory typing, and removes dead wrappers. Verified with go build ./..., go vet ./..., go test ./..., go test -race ./..., govulncheck ./..., codex-review, and autoreview.

[michiosw]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• fix(selfserve): harden managed observe cli readiness #286 👈 (View in Graphite)
• feat(observe): surface revoked install tokens; doctor managed-observe section #278
• feat(setup): kontext setup / --uninstall for self-serve managed observe #277
• feat(selfserve): scope-aware config/identity paths, user settings merge, version fallback #276
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[greptile-apps]

Greptile Summary

This PR hardens managed observe diagnostics and stream cursor persistence. The main changes are:

• Reports corrupt installation state separately from missing state.
• Returns and logs auth breadcrumb cleanup failures.
• Keeps managed stream cursor JSON compatible while using typed timestamps in memory.
• Removes unused managed config and installation helper wrappers.

Confidence Score: 5/5

This looks safe to merge.

• No blocking issues found in the changed code.

Important Files Changed

Filename	Overview
internal/managedstream/stream.go	Moves cursor parsing into state loading while preserving the existing persisted JSON shape.
internal/managedobserve/autherr.go	Makes breadcrumb deletion errors observable while keeping missing breadcrumbs as a no-op.
internal/managedobserve/doctor.go	Distinguishes missing installation identity from invalid or unreadable installation state.

_{Reviews (1): Last reviewed commit: "fix(selfserve): harden managed observe c..." | Re-trigger Greptile}