daily-issues-report: switch Copilot SDK driver from Python to Node in workflow runtime by Copilot · Pull Request #46894 · github/gh-aw

Copilot · 2026-07-20T17:11:26Z

The Daily Issues Report Generator has been failing repeatedly in the Copilot execution path, preventing daily report generation and leaving a persistent failing run in the workflow fleet. The failure came from the Python SDK driver path in the runtime/chroot environment.

Root cause addressed
- daily-issues-report was configured to use the Python Copilot SDK sample driver, which failed at runtime due to missing Python copilot module availability in the execution environment.
Workflow engine/runtime change
- Updated workflow engine driver in source workflow:
  - .github/workflows/daily-issues-report.md
- Repointed from Python driver to the Node SDK driver already used by other Copilot-SDK workflows.
Compiled workflow update
- Regenerated .github/workflows/daily-issues-report.lock.yml to reflect:
  - Node SDK install step (@github/copilot-sdk)
  - Node-based driver invocation inside the Copilot harness/chroot command path.

engine:
id: copilotcopilot-sdk: truedriver: .github/drivers/copilot_sdk_driver_sample_node.cjs

Fixes[deep-report] Fix recurring chroot-node-not-available failure in Daily Issues Report Generator #46869

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

pelikhan · 2026-07-20T17:50:26Z

@copilot investigate why python is not working

Copilot · 2026-07-20T18:02:02Z

@copilot investigate why python is not working

I investigated the failing runs for daily-issues-report (for example run 29753994707, failed agent job 88391655399). The Python path fails because the workflow invokes python3 .github/drivers/copilot_sdk_driver_sample_python.py, and inside the runtime it raises ModuleNotFoundError: No module named 'copilot' on every retry. That Python copilot SDK module is not present in this execution environment, so the driver cannot start. Commit 12eb056 addresses this by switching to the Node Copilot SDK driver path, which aligns with the available runtime/tooling.

pelikhan · 2026-07-20T18:33:58Z

@copilot update the compiler to Install the copilot python sdk

github-actions · 2026-07-20T18:53:58Z

🤖 PR Triage

Field	Value
Category	bug
Risk	🟡 medium
Priority Score	62/100
Recommended Action	`fast_track`

Score breakdown: impact 32 + urgency 20 + quality 10

Daily issues report is failing repeatedly due to Copilot Python SDK incompatibility. Switching to Node driver directly unblocks the daily report workflow. Draft — needs undraft + CI.

Generated by 🔧 PR Triage Agent · 34.3 AIC · ⌖ 7.71 AIC · ⊞ 5.6K · ◷

…vers Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot · 2026-07-20T19:07:23Z

@copilot update the compiler to Install the copilot python sdk

Implemented in ee58922. The compiler/runtime now installs the Python Copilot SDK into a workspace path (${GITHUB_WORKSPACE}/.gh-aw/copilot-sdk/python) and sets PYTHONPATH for Python copilot-sdk drivers so the module is available inside AWF/chroot execution.

github-actions · 2026-07-21T00:58:07Z

🔍 PR Triage — Run §29791496271

Field	Value
Category	bug
Risk	🟡 medium
Score	62/100 (impact 30 + urgency 20 + quality 12)
Action	⚡ `fast_track` (DRAFT — needs undraft)

Rationale: Daily issues report continuously failing due to Python SDK in runtime. Switch to Node fixes root cause. Undraft and merge to restore daily reporting.

Generated by 🔧 PR Triage Agent · 40.1 AIC · ⌖ 6.65 AIC · ⊞ 5.6K · ◷

pelikhan · 2026-07-21T03:38:02Z

@copilot merge main and recompile

…de-not-available Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

github-actions · 2026-07-21T03:58:19Z

🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅

github-actions · 2026-07-21T03:58:25Z

✅ PR Code Quality Reviewer completed the code quality review.

github-actions · 2026-07-21T03:58:48Z

✅ Design Decision Gate 🏗️ completed the design decision gate check.

No ADR enforcement needed: PR #46894 does not have the 'implementation' label and has 81 new lines of code in business logic directories (threshold: 100).

github-actions · 2026-07-21T03:58:51Z

✅ Test Quality Sentinel completed test quality analysis.

Copilot · 2026-07-21T03:59:27Z

@copilot merge main and recompile

Done. I merged main and recompiled; addressed in commits 506691450b (merge) and a74c9d6159 (recompile).

Copilot

Pull request overview

Switches Daily Issues Report to the Node Copilot SDK driver while adding isolated Python SDK installation support.

Changes:

Uses the Node SDK driver for the daily report.
Installs Python SDK dependencies into a dedicated target and configures PYTHONPATH.
Regenerates affected workflow lock files.

Show a summary per file

File	Description
`.github/workflows/daily-issues-report.md`	Selects the Node SDK driver.
`.github/workflows/daily-issues-report.lock.yml`	Applies the generated Node installation and invocation.
`.github/workflows/smoke-call-workflow.lock.yml`	Updates generated reusable-workflow inputs, permissions, and secrets.
`pkg/workflow/copilot_engine_installation.go`	Adds targeted Python SDK installation.
`pkg/workflow/copilot_engine_execution.go`	Configures Python SDK runtime discovery.
`pkg/workflow/copilot_engine_test.go`	Updates and extends SDK installation tests.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Files reviewed: 6/6 changed files
Comments generated: 3
Review effort level: Medium

+	if currentPythonPath, exists := env["PYTHONPATH"]; copilotSDKRuntimeID(workflowData) == "python" && (!exists || currentPythonPath == "") {
+ env["PYTHONPATH"] = copilotSDKPythonPathExpression
+	}


+ spec.command = workspaceCommandPrefix + fmt.Sprintf(
+ `mkdir -p "%[1]s" && python3 -m pip install --disable-pip-version-check --target "%[1]s" github-copilot-sdk==%[2]s`,
+ copilotSDKPythonTargetDir,
+ version,
+ )


 id: copilot
 copilot-sdk: true
-driver: .github/drivers/copilot_sdk_driver_sample_python.py
+driver: .github/drivers/copilot_sdk_driver_sample_node.cjs


github-actions

Skills-Based Review 🧠

Applied /tdd — commenting on test coverage gaps; no blocking issues.

📋 Key Themes & Highlights

Key Themes

Test coverage gaps: copilotSDKRuntimeID has TypeScript and Ruby extension branches that are not exercised by the new tests; the pre-existing PYTHONPATH guard path is also untested.
Correct fix: switching from Python to Node driver is the right call and directly addresses the root cause.

Positive Highlights

✅ New copilotSDKRuntimeID function cleanly centralises runtime detection in one place — good deep-module design.
✅ PYTHONPATH injection is properly scoped to Python-only drivers, avoiding unintended env pollution.
✅ Tests are structured clearly with named sub-tests following Arrange/Act/Assert.
✅ Compiled lock file and source workflow are updated together — no drift.

🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · 25.3 AIC · ⌖ 7.37 AIC · ⊞ 6.7K
_{Comment /matt to run again}

Comments that could not be inline-anchored

pkg/workflow/copilot_engine_execution.go:144

[/tdd] The copilotSDKRuntimeID switch covers .py, .ts/.mts, and .rb, but only .py (positive) and .cjs (node fallback) are exercised by the new tests — the TypeScript and Ruby branches have no coverage.

<details>
<summary>💡 Suggested additions to TestCopilotEngineCopilotSDKPythonDriverSetsPythonPath</summary>

Add sub-tests for the remaining extensions so regressions in those branches don't go undetected:

t.Run(&quot;typescriptdriverdoesnotsetPYTHONPATH&quot;, func(t*tes…
</details><details><summary>pkg/workflow/copilot_engine_execution.go:155</summary>**[/tdd]**The`PYTHONPATH`guardskipssettingthevariablewhenitisalreadynon-empty, butthere&#39;snotestcoveringthatpath — makingitunclearwhetheranexisting`PYTHONPATH`shouldbepreservedas-isorextended.
&lt;details&gt;
&lt;summary&gt;💡 Suggestedtest&lt;/summary&gt;
Addasub-testthatpre-populatestheenvmapwithanon-empty`PYTHONPATH`andassertstheexpectedoutcome (preserveorextend), makingthecontractexplicitandpreventingsilentregressions.
&lt;/details&gt;
@copilotpleaseaddr…
</details>

github-actions

Review: Python to Node.js SDK driver switch

Clean fix for the chroot Python module availability issue. Two complementary changes:

Immediate fix: daily-issues-report now uses the Node.js SDK driver, bypassing the Python availability problem entirely.
Systemic fix for Python users: --target install dir plus PYTHONPATH injection ensures Python SDK packages are accessible inside the chroot runtime, rather than installing to global site-packages.

The copilotSDKRuntimeID helper cleanly centralises runtime detection (command takes precedence over driver extension). Tests are thorough and updated correctly.

No blocking issues.

🧵 Reviewed using Impeccable skills by Impeccable Skills Reviewer · 35.2 AIC · ⌖ 5.47 AIC · ⊞ 5K

github-actions · 2026-07-21T04:03:43Z

🧪 Test Quality Sentinel Report

✅ Test Quality Score: 82/100 — Excellent

Analyzed 6 test(s): 6 design, 0 implementation, 0 violation(s).

📊 Metrics (6 tests)

Metric	Value
Analyzed	6 (Go: 6, JS: 0)
✅ Design	6 (100%)
⚠️ Implementation	0 (0%)
Edge/error coverage	3 (50%)
Duplicate clusters	1 (pre-existing table pattern)
Inflation	No (1.3:1 ratio)
🚨 Violations	0

Test	File	Classification	Issues
`TestCopilotEngineInstallationWithCommandAndCopilotSDK` (row: python command uses pip sdk install)	`copilot_engine_test.go:2194`	design_test	None
`TestCopilotEngineInstallationWithCommandAndCopilotSDK` (row: env wrapper command is detected)	`copilot_engine_test.go:2243`	design_test	None
`TestCopilotEngineInstallationWithCommandAndCopilotSDK` (row: custom command with firewall)	`copilot_engine_test.go:2253`	design_test	None
`TestCopilotEngineInstallationWithCopilotSDKDriver` (row: python driver uses pip sdk install)	`copilot_engine_test.go:2314`	design_test	None
`TestCopilotEngineCopilotSDKPythonDriverSetsPythonPath/python_driver_sets_PYTHONPATH`	`copilot_engine_test.go:2361`	design_test	None
`TestCopilotEngineCopilotSDKPythonDriverSetsPythonPath/node_driver_does_not_set_PYTHONPATH`	`copilot_engine_test.go:2385`	design_test	None

Verdict

✅ Passed. 0% implementation tests (threshold: 30%). No violations.

Score breakdown: design coverage (40/40) + edge coverage (15/30) + no duplicate penalty (-5 for 1 cluster) + no inflation (+10) = 82/100.

The new TestCopilotEngineCopilotSDKPythonDriverSetsPythonPath is well-structured: it pairs a positive assertion (Python driver emits PYTHONPATH) with a negative assertion (Node driver does not), covering the key behavioral invariant introduced by this PR. The four updated table rows correctly track the new --target install path.

References:

§29799900976

🧪 Test quality analysis by Test Quality Sentinel · 43.7 AIC · ⌖ 9.78 AIC · ⊞ 7K · ◷
_{Comment /review to run again}

github-actions

✅ Test Quality Sentinel: 82/100. 0% implementation tests (threshold: 30%). No violations. All 6 analyzed tests are design-level with correct build tags and no mock library usage.

github-actions

REQUEST_CHANGES — 3 blocking issues found

The core driver switch (Python → Node) for daily-issues-report is correct and the logic of the new copilotSDKRuntimeID helper is sound. However there are issues that need fixing before merge.

Blocking themes

PYTHONPATH silently dropped (copilot_engine_execution.go:154): the guard condition !exists || currentPythonPath == "" means any workflow that already has PYTHONPATH set skips adding the pip --target dir, breaking Python SDK imports at runtime.
Permissions escalation in smoke-call-workflow (smoke-call-workflow.lock.yml:89): pull-requests and issues promoted to write without documented rationale. Needs explicit justification, especially for a smoke workflow that may be triggered by untrusted PRs.
Implicit secrets regression (smoke-call-workflow.lock.yml:98): secrets: inherit replaced with an explicit 7-secret list — any secret the callee needs that is not in this list will silently arrive empty and cause auth failures. Needs verification against the callee's full secret usage.

Non-blocking: the two path constants (copilotSDKPythonPathExpression / copilotSDKPythonTargetDir) are a maintenance hazard worth addressing but not blocking.

🔎 Code quality review by PR Code Quality Reviewer · 58.4 AIC · ⌖ 4.93 AIC · ⊞ 5.6K
_{Comment /review to run again}

Comments that could not be inline-anchored

pkg/workflow/copilot_engine_execution.go:154

PYTHONPATH is silently dropped if already set: when a caller workflow already has PYTHONPATH set to a non-empty value, the pip --target directory is never appended, causing import github_copilot_sdk to fail at runtime with no clear error.

<details>
<summary>💡 Suggested fix</summary>

Always prepend unconditionally instead of skipping when the var exists:

ifcopilotSDKRuntimeID(workflowData) ==&quot;python&quot; {
existing:=env[&quot;PYTHONPATH&quot;]
ifexisting==&quot;&quot; {
env[&quot;P…
</details><details><summary>.github/workflows/smoke-call-workflow.lock.yml:89</summary>**Permissionsescalatedfromreadtowritewithoutjustification**: `pull-requests`upgradedfrom`read` → `write`and`issues: write` added. Thecommentsaysthesecomefromthecalledworkflow, butblindlyinheritingwritepermissionscreatesunnecessaryblastradiusifthecalledworkflowiscompromisedorhasabug.
&lt;details&gt;
&lt;summary&gt;💡 Whattoverify&lt;/summary&gt;
Auditwhat`smoke-workflow-call.lock.yml`actuallydoesat runtime. IfitonlyreadsPRs/issues (e.g. tofetchcontext) andwrit…
</details><details><summary>.github/workflows/smoke-call-workflow.lock.yml:98</summary>**Replacing`secrets: inherit`withanexplicitlistmaysilentlydroprequiredsecrets**: anysecretthecalledworkflowdependsonthatisnotinthislistwillarriveasanemptystringatruntime, producingauthfailureswithnoclearerror.
&lt;details&gt;
&lt;summary&gt;💡 Whattoverify&lt;/summary&gt;
Cross-checkthisexplicitlistagainstevery`secrets.`referencein`smoke-workflow-call.lock.yml`toconfirmnorequiredsecretis omitted. Theregressionriskishigh: `secrets: inherit`wasthesafed…
</details><details><summary>pkg/workflow/copilot_engine_execution.go:115</summary>**TwoconstantsencodethesameSDKinstallpathwithdifferentsyntaxes**: `copilotSDKPythonPathExpression` (execution.go) usesActionsexpressionsyntax`${{ github.workspace }}/...`, while`copilotSDKPythonTargetDir` (installation.go) usesshellsyntax`${GITHUB_WORKSPACE}/...`. Theymuststayinsyncmanually — renamingthepathinoneplacewon&#39;tupdatetheother.
&lt;details&gt;
&lt;summary&gt;💡 Suggestedfix&lt;/summary&gt;
Defineasinglesharedbaseconstantfortherelativepathsegment (`.gh-aw/cop…
</details>

Done

github-actions · 2026-07-21T08:28:27Z

🎉 This pull request is included in a new release.

Release: v0.82.15

Initial plan
8ddf00c

CopilotAI assigned Copilot and pelikhan Jul 20, 2026

CopilotAI linked an issue Jul 20, 2026 that may be closed by this pull request
[deep-report] Fix recurring chroot-node-not-available failure in Daily Issues Report Generator #46869
Closed

Copilotstarted work on behalf of pelikhan July 20, 2026 17:11 View session

github-actionsBot mentioned this pull request Jul 20, 2026
[Contribution Check Report] Contribution Check - 2026-07-20 #46693
Closed

Fix daily issues report Copilot SDK driver runtime
12eb056
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

CopilotAI changed the title ~~[WIP] Fix recurring chroot-node-not-available failure in Daily Issues Report Generator~~daily-issues-report: switch Copilot SDK driver from Python to Node in workflow runtimeJul 20, 2026

Copilotfinished work on behalf of pelikhan July 20, 2026 17:41

CopilotAI requested a review from pelikhan July 20, 2026 17:41

Copilotstarted work on behalf of pelikhan July 20, 2026 17:50 View session

Copilotfinished work on behalf of pelikhan July 20, 2026 18:02

Copilotstarted work on behalf of pelikhan July 20, 2026 18:34 View session

github-actionsBot added pr-action:fast_track pr-agent:copilot pr-priority:high pr-risk:medium pr-type:bug labels Jul 20, 2026

CopilotAIand others added 2 commits July 20, 2026 18:56

fix compiler/runtime python copilot sdk packaging for copilot-sdk dri…
876ccdf
…vers Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

refactor python copilot sdk install command generation and env guard
ee58922
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilotfinished work on behalf of pelikhan July 20, 2026 19:07

Copilotstarted work on behalf of pelikhan July 21, 2026 03:38 View session

Merge remote-tracking branch 'origin/main' into copilot/fix-chroot-no…
5066914
…de-not-available Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Recompile after merging main
a74c9d6
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

pelikhan marked this pull request as ready for review July 21, 2026 03:57

CopilotAI review requested due to automatic review settings July 21, 2026 03:57

Copilotstarted reviewing on behalf of pelikhan July 21, 2026 03:57 View session

Copilotfinished work on behalf of pelikhan July 21, 2026 04:00

CopilotAI reviewed Jul 21, 2026
View reviewed changes

github-actionsBot reviewed Jul 21, 2026
View reviewed changes

github-actionsBot mentioned this pull request Jul 21, 2026
[aw] No-Op Runs #43499
Closed

github-actionsBot approved these changes Jul 21, 2026
View reviewed changes

github-actionsBot previously requested changes Jul 21, 2026
View reviewed changes

pelikhan merged commit cf540d5 into mainJul 21, 2026
68 of 79 checks passed

pelikhan deleted the copilot/fix-chroot-node-not-available branch July 21, 2026 04:28

github-actionsBot mentioned this pull request Jul 21, 2026
Smoke Test: MAI-Code-1-Flash - 29812424728 #47025
Closed

Uh oh!

Conversation

CopilotAI commented Jul 20, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

pelikhan commented Jul 20, 2026

Uh oh!

CopilotAI commented Jul 20, 2026

Uh oh!

pelikhan commented Jul 20, 2026

Uh oh!

github-actionsBot commented Jul 20, 2026

🤖 PR Triage

Uh oh!

CopilotAI commented Jul 20, 2026

Uh oh!

github-actionsBot commented Jul 21, 2026

🔍 PR Triage — Run §29791496271

Uh oh!

pelikhan commented Jul 21, 2026

Uh oh!

github-actionsBot commented Jul 21, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actionsBot commented Jul 21, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actionsBot commented Jul 21, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actionsBot commented Jul 21, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

CopilotAI commented Jul 21, 2026

Uh oh!

CopilotAI left a comment

Choose a reason for hiding this comment

Pull request overview

Review details

Uh oh!

github-actionsBot left a comment

Choose a reason for hiding this comment

Skills-Based Review 🧠

Key Themes

Positive Highlights

Comments that could not be inline-anchored

Uh oh!

github-actionsBot left a comment

Choose a reason for hiding this comment

Review: Python to Node.js SDK driver switch

Uh oh!

github-actionsBot commented Jul 21, 2026

🧪 Test Quality Sentinel Report

Verdict

Uh oh!

github-actionsBot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actionsBot left a comment

Choose a reason for hiding this comment

REQUEST_CHANGES — 3 blocking issues found

Comments that could not be inline-anchored

Uh oh!

Uh oh!

github-actionsBot commented Jul 21, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

CopilotAI commented Jul 20, 2026•
edited
Loading

github-actionsBot commented Jul 21, 2026•
edited
Loading

github-actionsBot commented Jul 21, 2026•
edited
Loading

github-actionsBot commented Jul 21, 2026•
edited
Loading

github-actionsBot commented Jul 21, 2026•
edited
Loading