Skip to content

Track remaining PORT-010 exclusions upstream - #2298

Merged
randlee merged 1 commit into
fix/bg-6-sclint082-runnerfrom
fix/bg-6-sc081-4
Oct 30, 2026
Merged

randlee merged 1 commit into
fix/bg-6-sclint082-runnerfrom
fix/bg-6-sc081-4

Conversation

@randlee

@randlee randlee commented Oct 29, 2026

Copy link
Copy Markdown
Owner

Summary

sc-lint 0.8.2 still reports PORT-010 for all 18 paths when each exclusion is removed independently. Keep the exclusions while the upstream team classifies the cases, tracked in sc-lint #786; #720 covers the crate-root case only. Remove each exclusion when a released sc-lint fix covers its findings and the unsuppressed check passes.

Before and after the reason change, the published sc-portability check returned version 0.8.2, status pass, findings []. With each path individually unsuppressed at d70ff22, the exact emitted findings are captured in #786; counts:

  • crates/atm-core/src/atm_temp.rs: 4 PORT-010 finding(s)
  • crates/atm-core/src/atm_temp_sweeper.rs: 2 PORT-010 finding(s)
  • crates/atm-core/src/descriptor_limit.rs: 1 PORT-010 finding(s)
  • crates/atm-core/src/persistence.rs: 1 PORT-010 finding(s)
  • crates/atm-core/src/transfer_script.rs: 2 PORT-010 finding(s)
  • crates/atm-daemon-bootstrap/src/lib.rs: 2 PORT-010 finding(s)
  • crates/atm-daemon-bootstrap/src/protected_trust_root.rs: 2 PORT-010 finding(s)
  • crates/atm-daemon-client/src/lib.rs: 2 PORT-010 finding(s)
  • crates/atm-herdr/src/transport_socket.rs: 1 PORT-010 finding(s)
  • crates/atm-http-runtime/src/client.rs: 1 PORT-010 finding(s)
  • crates/atm-http-runtime/src/client_tail.rs: 3 PORT-010 finding(s)
  • crates/atm-http-runtime/src/http1_server.rs: 1 PORT-010 finding(s)
  • crates/atm-http-runtime/src/lib.rs: 3 PORT-010 finding(s)
  • crates/atm-http-runtime/src/loopback_tcp.rs: 2 PORT-010 finding(s)
  • crates/atm-http-runtime/src/runtime_listener.rs: 1 PORT-010 finding(s)
  • crates/atm-http-runtime/src/runtime_setup.rs: 1 PORT-010 finding(s)
  • crates/atm-http-runtime/src/unix_socket.rs: 17 PORT-010 finding(s)
  • crates/atm/src/main.rs: 1 PORT-010 finding(s)

Deliverables

  1. Probed all 18 exclusions, filed fix(daemon): preserve UDS guard cleanup on shutdown (DAEMON-SHUTDOWN-UDS-LEAK-AL11) #786 with exact findings and notified the sc-lint lead; updated .sc-lint/install.json reason with the open issue and release-based deletion condition, then regenerated kit-managed outputs.

Bead: atm-bg-6.sc081-4-r2-fix

Generated by arch-ctm@atm-dev codex:terra

@randlee
randlee added this pull request to stack #2290 October 29, 2026 22:05
@randlee

randlee commented Oct 29, 2026

Copy link
Copy Markdown
Owner Author

QA PASS: atm-bg-6.sc081-4-r2-fix round 2

Generated: 2026-10-29T22:07:48Z
QA bead: atm-bg-6.sc081-4-r2-qa (sprint bg-6)
Branch: fix/bg-6-sc081-4
Reviewed commit: 05b6e12151fb5ab773883a55bed0f4af7edaddb1
PR: #2298
Deliverables: 1/1
Findings filed: 0 blocking, 0 important, 0 minor; 0 closed as ceremony

Machine Status (JSON)

{
  "task": "atm-bg-6.sc081-4-r2-qa",
  "checked_bead": "atm-bg-6.sc081-4-r2-fix",
  "sprint": "bg-6",
  "round": 2,
  "branch": "fix/bg-6-sc081-4",
  "commit": "05b6e12151fb5ab773883a55bed0f4af7edaddb1",
  "verdict": "PASS",
  "deliverables": {"complete": 1, "total": 1 },
  "findings": {"blocking": 0, "important": 0, "minor": 0, "ceremony": 0 }
}

Findings

No new findings; locked SC081-4 verification passed.

Carried Findings

  • SC081-4 — fixed: .sc-lint/install.json:435-456 and sc-lint.toml:14-15 cite open sc-lint#786 and require removal when a released fix covers the path and its unsuppressed check passes.

Reviewers

Notes

git diff --check passed at the pinned head. Stack view could not locate a locally tracked matching stack because historical catalog worktrees are unavailable. The mandated gh pr checks 2298 --json name,state,bucket was blocked by the repository gh-wrapper as polling-prone; no CI status is claimed.

@randlee
randlee merged commit 6ffaa97 into integrate/phase-bg Oct 30, 2026
10 checks passed
@randlee
randlee deleted the fix/bg-6-sc081-4 branch October 30, 2026 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant