Fix Copilot SDK sample driver BYOK session configuration in Daily Model Inventory workflow

[Copilot]

Daily Model Inventory Checker failed because the Copilot SDK sample Node driver created a session without provider configuration, causing SDK auth initialization to fail (Session was not created with authentication info or custom provider) despite harness-resolved BYOK metadata.

• Root cause

  • The workflow uses .github/drivers/copilot_sdk_driver_sample_node.cjs.
  • Harness resolved and exported GH_AW_COPILOT_SDK_PROVIDER_BASE_URL, but the sample driver did not pass provider into createSession(...).

• What changed

  • Sample driver session config now includes BYOK provider when present
    • Added buildSessionConfig(...) in .github/drivers/copilot_sdk_driver_sample_node.cjs.
    • If GH_AW_COPILOT_SDK_PROVIDER_BASE_URL is set, session config now includes:
      • provider: { type: "openai", baseUrl: ... }
    • createSession(...) now consumes this config.
  • Driver module structure adjusted for safe import
    • Exported buildSessionConfig(...).
    • Wrapped execution with require.main === module so importing the driver does not execute main().

function buildSessionConfig(model, onPermissionRequest) {
  const config = { onPermissionRequest, model };
  const providerBaseUrl = process.env.GH_AW_COPILOT_SDK_PROVIDER_BASE_URL;
  if (providerBaseUrl) {
    config.provider = { type: "openai", baseUrl: providerBaseUrl };
  }
  return config;
}

[pelikhan]

@copilot delete this file

[Copilot]

Done in 573e0b7 — the file actions/setup/js/copilot_sdk_driver_sample_node.test.cjs has been deleted.

[Copilot]

Pull request overview

This PR updates the Copilot SDK sample Node driver used by the Daily Model Inventory workflow so it can correctly include BYOK provider configuration when the harness exports GH_AW_COPILOT_SDK_PROVIDER_BASE_URL, and so the driver can be safely imported without executing its main().

Changes:

• Added buildSessionConfig(model, onPermissionRequest) to include provider: { type: "openai", baseUrl } when GH_AW_COPILOT_SDK_PROVIDER_BASE_URL is set.
• Updated createSession(...) to consume the new session config builder.
• Wrapped execution with if (require.main === module) and exported buildSessionConfig for safe import.

Show a summary per file

File	Description
.github/drivers/copilot_sdk_driver_sample_node.cjs	Adds BYOK-aware session config construction and prevents side effects on import.

Copilot's findings

Tip

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

• Files reviewed: 1/1 changed files
• Comments generated: 1