Skip to content

fix\(app-control\): close the open threads pid-binding and eval follow-ups - part 2 of 4 #10400

Description

@melodic-automation-lanes

Part 2 of 4 of #9546, split at intake (owner Q93). The done-when below is copied from #9546 as written.

Done when

  • grep -nE '#[0-9]{3,}' over the changed skill files returns nothing.
  • grep -nE '#[0-9]{3,}' over each slice's changed skill files returns nothing (body criterion, every slice).
  • On macOS and Windows a threads capture records a blocked: line naming the platform as outside pid binding.
  • A bundled script under plugins/app-control/scripts/, with a test beside it, builds the shown script from the checked pid, uid, start time and hash; drive shows its full output before the yes.
  • The built script reads starttime from the fields after the last ) of /proc/<pid>/stat, and a test covers a comm with spaces and ).
  • The built script rechecks after the recorder exits and drops the capture on a difference; a test covers it.
  • The skill states that an unattended invalid pid gets blocked: launched process no longer matches before capture needs elevation.
  • An elevated attach records the attached pid's uid, start time and hash when the script is shown, and the script rechecks them.
  • The cpu and heap recorders that take a pid follow the same binding and recheck as threads.

Activity

  1. added
    needs-triageNot yet classified. Floor until a type and one priority tier are set.
    on Oct 27, 2026
  2. github-actions commented on Oct 27, 2026

    @github-actions
    Contributor

    Proposed work class: work-class: scoped
    Basis: automated intake triage; the lanes App applied this class.

    Size: Medium
    Complexity: Medium
    Uncertainty: Known
    Flags: none
    Blocked by: none

    Understood: this is part 2 of 4 of the split of #9546. It covers pid binding for the app-control threads, cpu and heap recorders. The work is a bundled script with tests, a recheck after the recorder exits, platform blocked: lines, and skill wording. The done-when items are concrete and checkable, so nothing is missing. Parts 1, 3 and 4 are sibling slices, not duplicates.

    Automated triage, applied by the lanes App.

  3. added
    priority: mediumReal value, no hard deadline; normal backlog flow.
    and removed
    needs-triageNot yet classified. Floor until a type and one priority tier are set.
    on Oct 27, 2026
  4. github-actions commented on Oct 27, 2026

    @github-actions
    Contributor

    The triage sweep dispatched intake triage for this issue (attempt 1 of 2).

  5. added
    status: readyTriaged, unblocked, and fully specified; eligible to pick up.
    work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.
    on Oct 27, 2026
  6. melodic-automation-lanes commented on Oct 27, 2026

    @melodic-automation-lanes
    ContributorAuthor

    Not admitted to the issue lane yet: blocker #10399 is not closed by a listed human or a merge.

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

    agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.priority: mediumReal value, no hard deadline; normal backlog flow.status: readyTriaged, unblocked, and fully specified; eligible to pick up.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.

    Type

    Fields

    No fields configured for Task.

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions