Skip to content

Session Handoff [auto-2157]: Drop the Path Argument From the VS Code Tasks Snippet's Mypy Swaps per #2157 #2211

Description

@ptr727

Next steps, in priority order

  1. Fix Drop the Path Argument From the VS Code Tasks Snippet's Mypy Swaps #2157 in catalog/snippets/configs/vscode-tasks-python.json: the header's two mypy swaps name a src path argument (uv run mypy src for a build repo whose CI checker is mypy, and uvx mypy src for the lint-only Scripts-profile form), where validate-task.yml's "Type check Python" step passes no target for a project configured at its own root, so a path overrides [tool.mypy] files and the local task checks a different file set than CI. Done looks like: both swaps name no path argument, and the "Adapt the target paths to the repo's package layout" line is reworded or dropped if it existed only to cover the src argument. Mirror the shape Drop the Path Argument From the Pre-Commit Snippet's Mypy Swap #2158 already applied to the pre-commit snippet's mypy swap.
  2. Run local-strict-review before every push, then open the pull request to develop carrying Closes on promotion: #2157 and drive it per drive-pr, scope develop.
  3. Close this lane out with a comment naming what merged, which issues it fixed, and anything filed along the way.

External blockers

None.

Internal dependencies

Step 2 follows step 1, and step 3 follows the develop merge. #2149 edits the same header (task names and stale CODESTYLE.md pointers) and #2025 covers the same uv run mypy src wording in python-codestyle. Do not fold either into this lane, and keep the change to the two mypy swap lines and the adapt line so a later lane on #2149 stays small.

State

Nothing started. No branch, worktree, or pull request exists for this lane. Re-derive develop's head, and whether any open pull request touches catalog/snippets/configs/vscode-tasks-python.json, before starting.

The parked decision queue

Count 0 for this lane.

What the last round did

An unattended picker created this lane from the open backlog. It changed no code.

What not to repeat

Nothing yet.

New learnings

None.

Activity

  1. added
    handoffA link in the session handoff chain, one open issue per track
    on Oct 1, 2026
  2. ptr727 commented on Oct 1, 2026

    @ptr727
    OwnerAuthor

    Parked: waiting on decision #2212.

    Done. Branch feature/auto-2157 is pushed at 2764d5c3200c86c4b7905a1668c06c4623bb8db4, three signed commits on develop 658034f7. It touches only catalog/snippets/configs/vscode-tasks-python.json:

    • Both mypy swaps name no path argument.
    • The Scripts-profile swap reads uvx mypy@latest / uvx pyright@latest.
    • "Adapt the target paths..." is replaced by a pointer line about a subdirectory listed in CI's python-directories input.

    Every commit carries a comment edit, so each was committed with PROSE_ALLOW_COMMENTS=1. The pull request needs the comments label when it opens.

    No pull request opened yet. The worktree /home/pieter/repos/worktrees/ProjectTemplate-auto-2157 is left standing, clean. Its local branch tracks origin/develop because of how it was created, so push it with an explicit refspec, git push origin HEAD:refs/heads/feature/auto-2157.

    Local review. Three passes are recorded. Pass 1 raised two findings: the missing @latest, fixed, and the dropped line still covering CI's root-config fallback for a listed subdirectory. Pass 2 raised two on the restored line: no project environment, and the undefined "declared". Pass 3 raised one: the pointer names validate-task.yml as if carried, where downstream repos reach it from the hub at a pinned SHA. That third finding is introduced and still open after two edit rounds, so editing stopped per local-strict-review "Disposing of Findings".

    Remains. Apply the option #2212 picks to line 17 of the snippet, run one more local strict review pass, then open the develop pull request carrying Closes on promotion: #2157 and the comments label, and drive it per drive-pr, scope develop. If the answer is option 2, decline pass 1's coverage finding as out of #2157's scope, or file it separately.

    Out of scope, as the handoff says. #2149 (task names, CODESTYLE.md pointers) and #2025 (uv run mypy src in python-codestyle) were not touched.

    No lesson issue was filed.

  3. added
    blockedBlocked on another issue: one in another repository, or a decision issue here for a parked handoff
    on Oct 1, 2026
  4. ptr727 commented on Oct 1, 2026

    @ptr727
    OwnerAuthor

    Handed back to the unattended loop by an attended session. Decision #2212 is answered: option 1, qualify the pointer so it names the hub's validate-task.yml "Type check Python step" at the commit this repository's caller workflow pins, then one more review pass and open the develop pull request. Follow the answer recorded on #2212.

  5. removed
    blockedBlocked on another issue: one in another repository, or a decision issue here for a parked handoff
    on Oct 1, 2026
  6. ptr727 commented on Oct 4, 2026

    @ptr727
    OwnerAuthor

    Closed out: #2368 merged to develop as a5cd7070e94a24577be8bfd20e459b532b38a615.

    What merged. catalog/snippets/configs/vscode-tasks-python.json only. Both mypy swaps name no path argument (uv run mypy, and uvx mypy@latest / uvx pyright@latest for the Scripts profile). The "Adapt the target paths..." line is replaced by the subdirectory pointer in the wording decision #2212 chose, option 1. The branch was merged up with develop first, which resolved a conflict with #2232's edit of the same header.

    Fixes. #2157, which stays open until the develop -> main promotion carries it, per the Closes on promotion line on #2368.

    Review. One local strict review pass on the final head raised nothing. Copilot raised no findings, plus one overview nit asking to split line 17, declined on the PR as a style preference no rule supports. CodeRabbit, prompted, raised one finding asking the pointer to list the step's working directory, config selection, and targets. It was declined, citing #2212, and CodeRabbit withdrew it.

    Filed along the way. Nothing. No lesson issue.

    Cleanup. The worktree, the local branch, and the remote branch feature/auto-2157 are deleted. The remote branch's tip was checked against the merged head first.

    #2149 and #2025 were not touched.

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

    handoffA link in the session handoff chain, one open issue per track

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions