Skip to content

Preserve original WSL extension help bytes in scratch-path exemptions #2200

Closed

Description

@Xuepoo

Priority: P1 | Area: runtime | Labels: fix,P1,area:runtime | Milestone: 0.0.24 | RFC: existing WSL path contract | Task: CTX-1306

The unpublished WSL scratch-path lint fix splits the extension help string with concat! and removes one leading space from two path-mapping lines. Compiling the exact, unmodified component_help_text function extracted from parent b99059bc proves both lines originally have two leading spaces. Parent help is 10,479 bytes; the unpublished patch produces 10,477 bytes.

Restore the missing spaces and correct the byte regression assertions while retaining the narrow, documented exemptions for intentional WSL guest paths. WSL path conversion and validation must remain unchanged.

Acceptance criteria:

  • Compile the exact parent function automatically extracted from Git, without editing its source, and capture its raw output.
  • Compare all help bytes and prove equality after the correction.
  • Preserve all WSL path conversion semantics and fail-closed validation.
  • Pass relevant help/component/WSL tests, repository gates, MSRV checking, and secret/diff checks; document genuine environment or pre-existing blockers.
  • Keep the patch limited to the two scoped source files, with portable evidence under ignored recording/ctx-1306/.

Implementation stops before push, pull request creation, or merge for independent commander review.

Activity

  1. added this to the 0.0.24 milestone on Oct 17, 2026
  2. Xuepoo commented on Oct 17, 2026

    @Xuepoo
    ContributorAuthor

    Delivered to main by the CTX-1317 integration merge efbe8f8 (PR #2209, squash). Source fix was independently approved (PX-6874 / PX-6883+PX-6884 / CTX-1311 review) and verified in the integrated head fd674a4 by independent integration review PX-6939 (severity none, all 43 manifest hashes verified, help-byte regression 0/101/0, real just ci-local both Quality shards passing 3569+3642). Content present on origin/main — verified by guard hermes-planner.

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

    P1Priority: higharea:runtimeArea: runtime orchestrationfixBug fix

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions