Skip to content

sessions: retire test-only try_ingest_source driver and its parse_new replacement namespacing #2738

Description

@ScriptedAlchemy

try_ingest_source (crates/tracedecay-sessions/src/runtime/source.rs) and the TranscriptSource::parse_new implementations it drives have no production caller: production catch-up is observation/port driven. Only test support (session_test_support.rs, transcript_ingest_suite/restart_atomicity.rs) reaches them. The Cursor and Codex parse_new paths still carry namespace_replacement_message_ids, a second message-id authority that production never uses.

After #2737 the scanner reports a replacement only for a genuine prefix rewrite, so this is not a live bug, but it is a parallel ingest authority kept alive by tests. Migrate those tests to the observation path, then delete the driver, the parse_new legacy paths and namespace_replacement_message_ids.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions