Skip to content

[Summary] Next issues and PRs to solve, prioritized #1064

@danielmeppiel

Description

@danielmeppiel

FIX-NOW execution status (board-authorized waves) -- WAVE 2 COMPLETE

Wave PR Status Notes
1 #635 CLOSED Author missed CLA window. Replaced by #1068.
1 #947 MERGED Lint + 3 Copilot findings + UX audit folded in single PR.
1 #991 MERGED Admin-override squash-merge. CI green.
1 #931 CLOSED-OBSOLETE Already on main via #1020.
1 #1055 MERGED Admin-override squash-merge. CI green.
1 #1065 MERGED Consolidated retro-fix (Copilot review on #1055/#991/#1065 itself).
2 #937 MERGED Maintainer-pushed fix: 2 remaining InstallScope.USER guards now allow absolutes; E2E asserts deployment (closes the test gap that let the bug ship). 6858 unit + 20 E2E pass.
2 #608 CLOSED-WONTFIX Stale author-branch merges deleted ~10 panel/agent infra files. Issue #605 stays valid but deprioritized.
2 #1068 MERGED Successor to #635: 15 open() sites across 5 modules get encoding="utf-8" + 9 CJK round-trip tests + Copilot review (raw CJK literals -> \uXXXX escapes per repo encoding rule) folded.

Cumulative outcome: 7 PRs merged (#947, #991, #1055, #1065, #937, #1068, plus tracked #931). 2 closed-obsolete/wontfix (#931, #608). 1 successor opened+merged (#1068).

Plan-v4 inputs captured for wave 3:

  1. Last-pusher rule (wave 1): orchestrator must NOT both push lint commits AND approve final SHA -- branch protection rejects with "Waiting on 1 reapproval". Board admin-merge is the only safe path when orchestrator pushes.
  2. Fold-not-followup (wave 1): Copilot review findings + UX-audit recommendations fold into the open PR (or one consolidated retro PR if source already merged).
  3. Copilot-review gate (wave 1): mandatory pre-merge for all PRs even when dissenting lens approves.
  4. Parallel UX subagent (wave 1): when a fix touches user-facing surface, spawn devx-ux-expert in parallel; findings fold into the same PR.
  5. Real-asset reproduction is required as merge-evidence (wave 2): unit/E2E tests can pass against broken behavior if they assert the wrong thing. The fix(install): allow local packages at --global scope; fix broken tests #937 deployment-vs-manifest gap is the canonical example.
  6. Maintainer-push when author silent (wave 2): for fork PRs where author is non-responsive, orchestrator pushes via gh-authenticated SSH to the fork (Co-authored-by credit). Worked on fix(install): allow local packages at --global scope; fix broken tests #937. Successor PR pattern (close + replace from microsoft/apm) only when fork-push impossible (the Standardize Unicode encodings to explicitly use UTF-8 #635 -> fix: extend explicit UTF-8 encoding to 5 remaining open() call sites #1068 path).

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/ci-cdGitHub workflows, merge queue, gh-aw integrations, release pipeline.status/triagedInitial agentic triage complete; pending maintainer ratification (silence = approval).type/automationAutomation script, workflow, gh-aw, dependabot config.

    Type

    No type

    Projects

    Status

    Done

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions