test: report terminated legacy session processes clearly

Analysed against e37664142 files changedby 2233admin
Open PR on GitHub

Change-risk score

8.5out of 10Typical
Repository health
3.9to4.8base against this head

This change sits in the middle third of this repository's own risk distribution, so it is about as risky as the work around it. It is riskier than 60% of this repository's own commits.

No changed file lost health in this diff. Nothing outside this PR depends on a contract it changed.

Files changed
2
in this diff
Contracts changed
0
removed or signature
Outside callers
0
not in this diff
Findings added
6
3 resolved

Blast radius on the repo map

Every file in the repository, grouped into its own directories and sized by lines. The ones this PR changes are lit, and everything that imports them is marked.

cratesdocsorchestrationprototy…skills/code-in…code-intel-cliplansevidenceschemassrctestsaudit_reportclimcp_serve
changed, health fellchanged, health rosechanged, health heldimports a changed fileuntouchedchanged by this PR

Drawing 574 of 748 files at a readable size, grouped by directory and sized by lines of code. 2 changed here reach 1 more by import. 174 are too small to see at this width, which is what the frame below is for.

Inside crates/code-intel-cli/tests

artifact_i…artifact_r…assistance…assistance…ast_grep_i…audit_repo…authority_transit…capability_exec.rscli_head_parity.rscodenexus…compatibi…compatibi…dag_coord…dag_run.rsdecision_gap.rsdecision_port.…decision_recor…deliver…doctor_…doctor_…env_contract…evidence_adm…evidence_pay…file_boundar…graph_adapte…hospital_dia…installer_versio…internalization_…mcp_ser…method_…method_…native_…pon_multila…ponytail_ga…primary_ent…project_ori…project_ori…repin.rsrepo…re…run_co…run_execut…runtime_ci…schema_lif…sentrux_ad…sentrux_an…se…ses…snapshot_i…span_apply.rsstaged_artifac…survival_scan.…toolcha…underst…ver…

61 files, 1 changed by this PR and 0 that import one. Click any directory above to frame it instead.

Changed

What this PR added, and what it took away

Introduced findings are the ones inside the lines this PR added, not the file's existing ones. The resolved column is the only place in the product that lists them.

Introduced (6)
  • lowprimitive obsession
    crates/code-intel-cli/tests/primary_entry/legacy_session.rs:54

    legacy_session_failure_message takes 6 parameters

  • lowerror handling
    crates/code-intel-cli/tests/primary_entry/legacy_session.rs:16

    unwrap/expect turns a recoverable error into a crash

  • lowerror handling
    crates/code-intel-cli/tests/primary_entry/legacy_session.rs:30

    unwrap/expect turns a recoverable error into a crash

  • lowerror handling
    crates/code-intel-cli/tests/primary_entry/legacy_session.rs:43

    unwrap/expect turns a recoverable error into a crash

  • lowerror handling
    crates/code-intel-cli/tests/primary_entry/legacy_session.rs:99

    panic!/unreachable!/todo!/unimplemented! aborts the process unconditionally

  • lowerror handling
    crates/code-intel-cli/tests/primary_entry/legacy_session.rs:105

    panic!/unreachable!/todo!/unimplemented! aborts the process unconditionally

Resolved (3)
  • function hotspot
    crates/code-intel-cli/tests/primary_entry.rs
  • dry violation
    crates/code-intel-cli/tests/primary_entry.rs
  • error handling
    crates/code-intel-cli/tests/primary_entry.rs

Who else knows this code

The primary author of each changed file by share of its recent commits, mined from git history. Advisory: it says who has context, not who must review.

  • crates/code-intel-cli/tests/primary_entry.rs2233admin 92% of 9 commits · the PR author

Get this on your own pull requests

This page came from an index of 2233admin/code-intel-pipeline. No model calls, no configuration, and it refreshes on every push. Install the bot and every pull request gets one of these.