Skip to content

Step 7 TDD only in this exact registered worktree. Add failing-first tests, primarily tests/typed_ooda_actor_session_scope.rs plus focused cfg(test) modules only where private access is unavoidable. C #5004

Description

@rysweet

Task Description

Step 7 TDD only in this exact registered worktree. Add failing-first tests, primarily tests/typed_ooda_actor_session_scope.rs plus focused cfg(test) modules only where private access is unavoidable. Cover: goal_slug normalized-equivalence fixed vectors; authenticated canonical six-field ActorScopeKey validation/encoding/fingerprint fixed vectors; one resolved immutable session correlation across src/ooda_actions/advance_goal/typed_goal_session.rs, src/typed_ooda/route.rs, src/typed_ooda/executor.rs and terminal access; compatible legacy reuse; incompatible legacy scoped isolation; scoped collision/corruption; malformed scope/persisted inputs; direct incompatible rebinding including goal mismatch; concurrent compatible/incompatible resolution; resolution telemetry exact outcome allowlist and prohibited-field exclusion. Preserve the three staged documentation files. Do not implement production behavior. Tests must compile and execute, then fail only at named assertions for missing Step 8 behavior. Run cargo fmt --all and NODE_OPTIONS=--max-old-space-size=32768 cargo test --test typed_ooda_actor_session_scope -- --nocapture. Return the changed test/scaffolding paths and exact expected failures.

Requirements

✓ Set NODE_OPTIONS=--max-old-space-size=32768

Final unambiguous requirements

No blocking ambiguity remains. Apply these decisions during design:

  • Work exclusively in the current registered worktree.
  • Add failing-first Step 7 tests centered on tests/typed_ooda_actor_session_scope.rs.
  • Add narrowly scoped in-file #[cfg(test)] modules only for behavior inaccessible through public APIs.
  • Do not alter production behavior, persisted formats, or public APIs.
  • Preserve all currently staged documentation files unchanged. Although the request mentions three, the repository currently has seven; the stricter preservation rule applies to all seven.
  • Use literal, independently predetermined fixed vectors for:
    • normalized-equivalent goal_slug values;
    • canonical six-field ActorScopeKey encoding;
    • scope fingerprints.
  • Cover every specified scenario: immutable end-to-end correlation, legacy reuse/isolation, collisions, corruption, malformed inputs, incompatible rebinding including goal mismatch, concurrency, and telemetry.
  • Telemetry tests must assert an exact outcome allowlist and explicitly reject prohibited or sensitive fields.
  • Deliberate Step 8 gaps must be isolated behind uniquely named assertions. Compilation, fixture setup, and incidental assertions must pass before those assertions fail.
  • The targeted test command is expected to exit unsuccessfully solely because of those named failing-first assertions:
    NODE_OPTIONS=--max-old-space-size=32768 cargo test --test typed_ooda_actor_session_scope -- --nocapture
  • Run cargo fmt --all.
  • Final delivery must list every changed test/test-only scaffolding path and reproduce the exact intentional failure names or messages.

Acceptance Criteria

  • All explicit requirements met
  • Tests passing
  • Philosophy compliant
  • Documentation updated

Classification

Generated via default-workflow recipe

Metadata

Metadata

Assignees

No one assigned

    Labels

    workflow:defaultCreated by default-workflow recipe

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions