8000 T51 — Move the exec and RPC-crate tests into their own files · Issue #2376 · gominimal/minimal · GitHub
[go: up one dir, main page]

Skip to content

T51 — Move the exec and RPC-crate tests into their own files #2376

Closed

Description

@mitodrummer

Task

slice size difficulty model who estimated lines
Retained sessions M routine standard an agent 4220 (4100 code, 120 tests)

Exec and wire tests pass from their own files.

Covers:

  • none

Layers:

  • exec:
    • crates/minimald/src/exec.rs: mod tests
    • crates/minimald/src/exec/tests.rs (new): moved
    • cargo nextest run -p minimald without_an_exporter_nothing_is_recorded
  • wire:
    • crates/minimald-rpc/src/lib.rs: mod tests
    • crates/minimald-rpc/src/tests.rs (new): moved
    • cargo nextest run -p minimald-rpc package_check_skipped_is_omitted_when_false
Plan block
### T51 — Move the exec and RPC-crate tests into their own files
  slice: S0
  kind: enablement
  covers: none
  size: M
  est: 4100 code, 120 tests
  executor: agent
  difficulty: routine
  model: standard
  diagnostics: none: a pure move, no behaviour
  observability: none: a pure move, no signal
  layers:
    - exec:
      - crates/minimald/src/exec.rs: mod tests
      - crates/minimald/src/exec/tests.rs (new): moved
      test: cargo nextest run -p minimald without_an_exporter_nothing_is_recorded
    - wire:
      - crates/minimald-rpc/src/lib.rs: mod tests
      - crates/minimald-rpc/src/tests.rs (new): moved
      test: cargo nextest run -p minimald-rpc package_check_skipped_is_omitted_when_false
  produces: none
  consumes: none
  proof: Exec and wire tests pass from their own files.

Activity

  1. added
    kind:taskA task the plan skill rendered; the body carries ## Task
    size:M200 to 500 net lines; one PR
    model:standardA standard model: work that needs care, or routine work that is large
    plan:enablementUnblocks other tasks: a lane, a gated path, a missing piece of the tree
    on Oct 19, 2026
  2. minimal-foundry commented on Oct 20, 2026

    @minimal-foundry
    Contributor

    Done. #2451 merged.

    branch agent/2376-move-the-exec-and-rpc-crate-tests-into-their-own · epic #2236 · PR #2451

    round outcome job
    1 changed nothing: Revise round 1: this round adds no commits. The directive gave no failure text, because the runtime could not read the job logs (token mint refused, HTTP 422). I found no defect in the two commits on 83c2b1d1
    2 changed nothing: Revise round 2: no change made. The directive lists four failed checks (ci-macos-success, e2e, ci-linux-native-success, tests), but every job log was unreadable (token mint refused), so no failure tex ad0b8e07
    3 pushed 44ff5484
    1 pushed 183cf897
    1 changed nothing: Revise round 1: no code change this round. The one open thread asks to keep tokio::spawn and the #[expect(clippy::disallowed_methods)] in the move. That change already has its own commits, each with a ddb6f335
    2 pushed 337f0f29

    Updated 2026-10-20T23:30:42.869Z.

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

    kind:taskA task the plan skill rendered; the body carries ## Taskmodel:standardA standard model: work that needs care, or routine work that is largeplan:enablementUnblocks other tasks: a lane, a gated path, a missing piece of the treesize:M200 to 500 net lines; one PR

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      0