Skip to content

fix: add missing formatItem and program imports in cli.test.ts#1265

Merged
lpcox merged 2 commits intoclaude/add-copilot-api-target-flagfrom
copilot/66859677809-fix-github-actions-workflow
Mar 12, 2026
Merged

fix: add missing formatItem and program imports in cli.test.ts#1265
lpcox merged 2 commits intoclaude/add-copilot-api-target-flagfrom
copilot/66859677809-fix-github-actions-workflow

Conversation

Copy link
Contributor

Copilot AI commented Mar 12, 2026

TypeScript type check CI was failing because formatItem and program were exported from cli.ts but omitted from the import statement in cli.test.ts.

Change

Added the two missing names to the existing import:

// before
import { ..., emitApiProxyTargetWarnings } from './cli';

// after
import { ..., emitApiProxyTargetWarnings, formatItem, program } from './cli';
Original prompt

Fix the failing GitHub Actions workflow TypeScript Type Check
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Job ID: 66859677809
Job URL: https://github.com/github/gh-aw-firewall/actions/runs/23021761323/job/66859677809


💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Copilot AI changed the title [WIP] [66859677809] Fix failing GitHub Actions workflow TypeScript type check fix: add missing formatItem and program imports in cli.test.ts Mar 12, 2026
Copilot AI requested a review from lpcox March 12, 2026 20:19
@lpcox lpcox marked this pull request as ready for review March 12, 2026 20:23
Copilot AI review requested due to automatic review settings March 12, 2026 20:23
@lpcox lpcox merged commit f05f453 into claude/add-copilot-api-target-flag Mar 12, 2026
@lpcox lpcox deleted the copilot/66859677809-fix-github-actions-workflow branch March 12, 2026 20:23
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a TypeScript type check CI failure by adding two missing imports (formatItem and program) to cli.test.ts. These symbols are exported from cli.ts and used in the test file but were not included in the import statement.

Changes:

  • Added formatItem and program to the import statement in cli.test.ts to resolve the TypeScript compilation error.

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

You can also share your feedback on Copilot code review. Take the survey.

lpcox added a commit that referenced this pull request Mar 12, 2026
…c endpoint detection (#1264)

* Initial plan

* feat(proxy): add validation for --copilot-api-target flag

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* test: add docker-manager tests for COPILOT_API_TARGET

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* feat(proxy): handle GHEC domains in deriveCopilotApiTarget

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* Potential fix for pull request finding 'CodeQL / Log injection'

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>

* docs: add enterprise configuration guide for GHEC/GHES

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* fix: add missing formatItem and program imports in cli.test.ts (#1265)

* Initial plan

* fix: add missing formatItem and program imports in cli.test.ts

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

* fix: drop -f from curl to avoid GitHub API rate-limit flakiness (#1267)

* Initial plan

* fix: drop -f from curl to avoid GitHub API rate-limit flakiness

Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>

---------

Co-authored-by: anthropic-code-agent[bot] <242468646+Claude@users.noreply.github.com>
Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com>
Co-authored-by: Landon Cox <landon.cox@microsoft.com>
Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants