Skip to content

apr ptx --help (0.69.3) omits the 'unavailable in this build' marker that #2399 says it must carry #4474

Closed as not planned

Description

@noahgift

Found while building MEGA-001 demo D11 (refusals shown by name), with apr 0.69.3 (apr 0.69.3 (v0.69.3+no-git), forjar-declared course-bin copy on lambda).

Observed. The top-level listing marks the command:

$ apr --help | grep -A1 ' ptx '
  ptx                      PTX analysis and bug detection [unavailable in this build: cargo install
                           aprender --features ptx]

but the leaf help does not:

$ apr ptx --help | head -4
PTX analysis and bug detection (register pressure, roofline)

#2399 finding 1: on a build without the analyzer this line is the only thing a user sees before
running the command, so it has to say so.

Running it refuses correctly (exit 9, error: Feature not enabled: apr ptx needs the PTX analyzer... cargo install aprender --features ptx).

Expected. apr ptx --help's first line carries the same [unavailable in this build: …] marker, as its own long help (citing #2399 finding 1) says it must. A tool that probes availability via <verb> --help (as demo-kit's preflight does) currently sees ptx as available.

Falsifier. On a build without --features ptx: apr ptx --help | head -1 | grep -q 'unavailable in this build' must succeed.

Not fixed on our side (MEGA-001 rule: apr defects are filed, not patched).

Activity

  1. added this to the 0.71.0 milestone on Oct 8, 2026
  2. noahgift commented on Oct 12, 2026

    @noahgift
    ContributorAuthor

    Status (owner aprender-wcensus): root cause found, fix is small.

    crates/apr-cli/src/extended_commands.rs: when trueno-explain is off, the cfg_attr sets only about, the short help that apr --help lists. The variant's multi-line doc comment still supplies long_about, and apr ptx --help prints long_about. So the leaf help loses the marker and instead shows the internal "#2399 finding 1" maintainer note.

    Fix: in the same not(trueno-explain) cfg_attr, also set long_about to the marker line plus the description. Move the maintainer note to a // comment so it never reaches help text. Add a test in cli_commands that the first line of apr ptx --help carries [unavailable in this build: when the feature is off. A PR follows when a slot opens.

  3. added
    P1High priority
    and removed
    P0Critical priority
    on Oct 14, 2026
  4. noahgift commented on Oct 14, 2026

    @noahgift
    ContributorAuthor

    P0 to P1 (2026-10-14 P0 triage): P0 now means it blocks the next release or carries the operator's own words on it; this issue does neither, so it stays on the 0.71.0 milestone at P1.

  5. noahgift commented on Oct 14, 2026

    @noahgift
    ContributorAuthor

    Folded into #5309 as a checklist row (TRIAGE-100, 2026-10-14): the open-ticket cap is 100, so this ticket is now tracked on that master and closed as not planned. Nothing is lost: reopen it when its work resumes, and an open PR for it reopens it.

    Agent: aprender-clerk

  6. noahgift commented on Oct 14, 2026

    @noahgift
    ContributorAuthor

    Row moved: #5309 was split into named masters (TRIAGE-100 master naming), so this ticket's checklist row is now on #5328.

    Agent: aprender-clerk

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

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions