Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 24, 2025

Success criteria

  • CodeQL workflow executes without build-mode errors
  • ESM build analyzed by CodeQL
  • Format pipeline check passes

How to test

  1. Push changes to trigger CodeQL workflow on PR
  2. Verify workflow completes successfully
  3. Confirm CodeQL results show analysis of dist/webchat.esm.js
  4. Run npm run prettier:check to verify formatting compliance

Security

  • No security implications

Additional considerations

  • This PR might have performance implications

Documentation Considerations

No documentation changes required. Internal CI/CD workflow configuration only.


Changes

JavaScript/TypeScript does not support manual build mode in CodeQL. Changed to none and rely on pre-build step:

- language: javascript-typescript
  build-mode: none  # was: manual

Applied prettier formatting to CodeQL configuration files per repository standards.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: kwinto <90881+kwinto@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Nov 24, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • download.cypress.io
    • Triggering command: /usr/local/bin/node node index.js --exec install (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Add new GitHub Actions workflow for CodeQL analysis Fix CodeQL workflow build-mode error and formatting Nov 24, 2025
Copilot AI requested a review from kwinto November 24, 2025 15:45
@kwinto kwinto marked this pull request as ready for review November 24, 2025 16:11
@kwinto kwinto merged commit a99703c into kwinto-patch-1 Nov 24, 2025
3 checks passed
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.

2 participants