ci(test): stop the v1 monolith re-running reborn composition/memory/secrets

The legacy `tests` job re-ran three things already covered by reborn-side
CI:
- `cargo test -p ironclaw_reborn_composition --features libsql,postgres`
- `cargo test -p ironclaw_memory`
- the `secret-store-contract-tests` job (`-p ironclaw_secrets --test
  secret_store_contract`)

All three run per-crate in reborn-tests.yml (libsql) and in coverage.yml
(`cargo llvm-cov --all-features --workspace` with a Postgres service), so
the v1 monolith was duplicating them on every legacy run. Remove the two
in-job steps and the standalone secret-store job (it was not gated by the
run-tests aggregator). Pure waste removal — no coverage lost: libsql via
the closure, postgres via coverage.yml + the hooks-parity Postgres job.

Scope note: this does NOT touch the 31 reborn_*.rs root tests the v1
`cargo test` also re-runs. Excluding those cleanly needs binary-level
exclusion (the obvious `--skip reborn_` is unsafe — the v1 suite has its
own reborn_*-named tests in dockerfile_runtime_home.rs / workspace), so
that is deferred to the v1 build-smoke restructure.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: serrrfirat

.github/workflows/test.yml

@@ -199,48 +199,12 @@ jobs:
         run: |
           timeout --signal=INT --kill-after=30s 40m \
             cargo test ${{ matrix.flags }} -- --nocapture
-      # The root cargo test invocation above runs only the `ironclaw`
-      # package's tests. Reborn memory regression coverage lives in
-      # `crates/ironclaw_memory/tests/*` and must be explicitly invoked
-      # with `-p ironclaw_memory` (or `--workspace`) — otherwise the
-      # Tier A guards for PR #3180 invariants compile but never run.
-      - name: Run reborn composition substrate tests
-        run: |
-          timeout --signal=INT --kill-after=30s 15m \
-            cargo test -p ironclaw_reborn_composition --no-default-features --features libsql,postgres --tests -- --nocapture
-
-      - name: Run ironclaw_memory tests
-        run: |
-          timeout --signal=INT --kill-after=30s 15m \
-            cargo test -p ironclaw_memory --tests -- --nocapture
-
-  secret-store-contract-tests:
-    name: Secret Store Contract Tests
-    needs: changes
-    if: needs.changes.outputs.docs_only != 'true' && needs.changes.outputs.has_legacy_tests == 'true' && needs.changes.outputs.has_core_code == 'true'
-    runs-on: ubuntu-latest
-    timeout-minutes: 15
-    env:
-      CARGO_PROFILE_DEV_DEBUG: 0
-      CARGO_PROFILE_TEST_DEBUG: 0
-    steps:
-      - name: Checkout repository
-        uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6
-        with:
-          ref: ${{ inputs.ref || github.sha }}
-          persist-credentials: false
-      - name: Install Rust
-        uses: dtolnay/rust-toolchain@29eef336d9b2848a0b548edc03f92a220660cdb8 # stable
-      - uses: Swatinem/rust-cache@e18b497796c12c097a38f9edb9d0641fb99eee32 # v2
-        with:
-          key: secret-store-contract
-          # Keep saves to protected-branch and merge_group runs so the ~10 GB
-          # repo cache LRU is seeded by shared states, not arbitrary PR branches.
-          save-if: ${{ (github.event_name == 'push' && github.ref == 'refs/heads/main') || github.event_name == 'merge_group' }}
-      - name: Run secret store contract
-        run: |
-          timeout --signal=INT --kill-after=30s 10m \
-            cargo test -p ironclaw_secrets --test secret_store_contract -- --nocapture
+      # Note: ironclaw_reborn_composition, ironclaw_memory, and the
+      # ironclaw_secrets secret_store_contract are no longer re-run here.
+      # They are covered per-crate by reborn-tests.yml (libsql) and by
+      # coverage.yml (`cargo llvm-cov --all-features --workspace` with a
+      # Postgres service), so running them in the v1 monolith was pure
+      # duplication.
 
   heavy-integration-tests:
     name: Heavy Integration Tests