Skip to content

chore(deps): bump globals 17.7→17.8 and markdownlint-cli2 0.23.1→0.23.2 - #6789

Merged
lpcox merged 3 commits into
mainfrom
copilot/deps-safe-updates-2026-07-31
Jul 31, 2026
Merged

chore(deps): bump globals 17.7→17.8 and markdownlint-cli2 0.23.1→0.23.2#6789
lpcox merged 3 commits into
mainfrom
copilot/deps-safe-updates-2026-07-31

Conversation

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Patch-level dependency updates with 0 audit vulnerabilities.

Changes

  • globals: 17.7.017.8.0
  • markdownlint-cli2: 0.23.10.23.2

Both package.json and package-lock.json updated via npm install.

Copilot AI linked an issue Jul 31, 2026 that may be closed by this pull request
3 tasks
Copilot AI changed the title [WIP] Update safe patch-level dependencies chore(deps): bump globals 17.7→17.8 and markdownlint-cli2 0.23.1→0.23.2 Jul 31, 2026
Copilot finished work on behalf of lpcox July 31, 2026 05:38
Copilot AI requested a review from lpcox July 31, 2026 05:38
@lpcox
lpcox marked this pull request as ready for review July 31, 2026 13:39
Copilot AI review requested due to automatic review settings July 31, 2026 13:39

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates development dependencies for linting and global definitions.

Changes:

  • Updates globals from 17.7.0 to 17.8.0.
  • Updates markdownlint-cli2 and related transitive dependencies.
  • Refreshes lockfile integrity metadata.
Show a summary per file
File Description
package.json Updates direct development dependencies.
package-lock.json Locks updated direct and transitive packages.

Review details

Tip

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

  • Files reviewed: 1/2 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread package.json Outdated
"eslint-plugin-security": "^3.0.1",
"glob": "^13.0.6",
"globals": "^17.7.0",
"globals": "^17.8.0",
@lpcox

lpcox commented Jul 31, 2026

Copy link
Copy Markdown
Collaborator

@copilot address review feedback

Copilot AI commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

@copilot address review feedback

Reverted globals back to ^17.7.0 in both package.json and package-lock.json — the spec and the resolved version are both 17.7.0 now. The PR now contains only the markdownlint-cli2 0.23.10.23.2 patch bump as originally described. Done in commit chore(deps): revert globals to 17.7.0; keep only markdownlint-cli2 patch bump.

Copilot finished work on behalf of lpcox July 31, 2026 14:00
@github-actions

Copy link
Copy Markdown
Contributor

✅ Copilot review passed with no inline comments.

@copilot Add the ready-for-aw label to this PR to trigger agentic CI smoke tests.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 98.89% 98.92% 📈 +0.03%
Statements 98.81% 98.83% 📈 +0.02%
Functions 99.20% 99.20% ➡️ +0.00%
Branches 95.27% 95.27% ➡️ +0.00%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/log-directory-setup.ts 96.2% → 100.0% (+3.78%) 96.3% → 100.0% (+3.71%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Smoke Copilot BYOK completed. Copilot BYOK mode operational. 🔓

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

📰 VERDICT: Smoke Copilot has concluded. All systems operational. This is a developing story. 🎤

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Smoke Copilot BYOK AOAI (api-key) completed. Copilot AOAI BYOK (api-key) mode operational. 🔓

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Build Test Suite completed successfully!

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Smoke Gemini completed. All facets verified. 💎

Gemini smoke test completed with failures.

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

📡 Smoke OTel Tracing completed. All tracing scenarios validated. ✅

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Smoke Copilot BYOK AOAI (Entra) completed. Copilot AOAI BYOK (Entra) mode operational. 🔓

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

🛡️ Smoke Copilot Network Isolation confirmed the egress allowlist is enforced. ✅

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Chroot tests passed! Smoke Chroot - All security and functionality tests succeeded.

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

📰 VERDICT: Smoke Docker Sbx has concluded. All systems operational. This is a developing story. 🎤

@github-actions

github-actions Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Smoke Claude passed

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Copilot BYOK (Direct) ✅

Status: PASS — Direct BYOK mode (COPILOT_PROVIDER_API_KEY) working correctly via api-proxy sidecar.

🔑 BYOK report filed by Smoke Copilot BYOK
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Copilot Engine@lpcox

  • ✅ GitHub MCP connectivity (list_pull_requests succeeded)
  • ✅ GitHub.com connectivity (HTTP 200)
  • ✅ File write/read test passed

Pre-fetched PRs: #6798 "Upgrade gh-aw extension to latest pre-release", #6780 "Validate bounded-query sandbox runtime matrix"

Overall: PASS

📰 BREAKING: Report filed by Smoke Copilot
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Services Connectivity — FAIL

  • Redis: ❌ Could not connect to Redis at host.docker.internal:6379: Temporary failure in name resolution
  • PostgreSQL pg_isready: ❌ host.docker.internal:5432 - no response
  • PostgreSQL SELECT 1: ❌ could not translate host name "host.docker.internal" to address: Temporary failure in name resolution

Overall: FAILhost.docker.internal DNS resolution failed inside the AWF sandbox.

🔌 Service connectivity validated by Smoke Services
Add label ready-for-aw to run again

@github-actions github-actions Bot added the smoke-copilot-network-isolation Copilot network-isolation egress smoke test label Jul 31, 2026
@github-actions

Copy link
Copy Markdown
Contributor

@lpcox Smoke test: Copilot network isolation egress enforcement

EGRESS_RESULT allow=pass deny=pass

✅ Allowed domain (github.com) reachable: allowed=200
✅ Blocked domain (example.com) denied: CONNECT tunnel failed, response 403

Overall status: PASS

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • example.com

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "example.com"

See Network Configuration for more information.

🛡️ Egress verdict from Smoke Copilot Network Isolation
Add label ready-for-aw to run again

@github-actions github-actions Bot mentioned this pull request Jul 31, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Smoke test results:

  • Upgrade gh-aw extension to latest pre-release
  • Validate bounded-query sandbox runtime matrix
  • GitHub PR review: ✅
  • Safe inputs GH CLI: ❌ (safeinputs-gh unavailable; used gh fallback)
  • Playwright title check: ✅
  • File write/read: ✅
  • Discussion lookup: ✅ (commenting unavailable in this workflow)
  • Build: ✅
    Overall: FAIL

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

  • registry.npmjs.org

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
  allowed:
    - defaults
    - "registry.npmjs.org"

See Network Configuration for more information.

🔮 The oracle has spoken through Smoke Codex
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Claude Engine Validation

Check Result
API status ✅ PASS
GH check ✅ PASS
File status ✅ PASS

Overall result: PASS

Generated by Smoke Claude for #6789 · haiku45 · 54.9 AIC · ⊞ 3.6K ·
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

Gemini Smoke Test Results

Overall Status: FAIL

💎 Faceted by Smoke Gemini
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

Chroot Version Comparison

Runtime Host Version Chroot Version Match?
Python Python 3.12.13 Python 3.12.13 ✅ YES
Node.js v24.18.0 v22.23.1 ❌ NO
Go go1.22.12 go1.22.12 ✅ YES

Result: Not all versions matched (Node.js differs between host and chroot environment). smoke-chroot label not applied.

Tested by Smoke Chroot
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

📡 Smoke Test: API Proxy OTel Tracing — Results

# Scenario Result
1 Module Loading otel.js loads; isEnabled()true; exports startRequestSpan, setTokenAttributes, setBudgetAttributes, endSpan, endSpanError, shutdown, isEnabled, ...
2 Test Suite ✅ 59/59 tests passed (2 suites: otel.test.js, otel-fanout.test.js) covering span creation, gen_ai.usage.* attributes, OTLP export (fan-out + single endpoint), FileSpanExporter, and error handling
3 Env Var Forwarding ⚠️ OTEL_EXPORTER_OTLP_ENDPOINT / GITHUB_AW_OTEL_TRACE_ID are not in src/services/api-proxy-service.ts directly, but are forwarded via src/services/api-proxy-env-config.ts (which that file delegates to). Functionally correct; the smoke-test grep target is stale relative to a prior refactor into a dedicated env-config module.
4 Token Tracker Integration onUsage callback present in token-tracker-http.js (invoked after normalized usage extraction, used as the OTEL hook point)
5 OTEL Diagnostics i️ otel.jsonl contains only gh-aw workflow-level spans (gh-aw, gh-aw.agent.setup); no api-proxy request spans, since this smoke run made no actual LLM API calls through the sidecar. Export path itself is validated by scenario 2's unit tests.

Overall: Pass, with one minor documentation/check drift noted in Scenario 3 (recommend updating the smoke test's grep path to include api-proxy-env-config.ts).

📡 OTel tracing validated by Smoke OTel Tracing
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

🏗️ Build Test Suite Results

Ecosystem Project Build/Install Tests Status
Bun elysia 1/1 passed ✅ PASS
Bun hono 1/1 passed ✅ PASS
C++ fmt N/A ✅ PASS
C++ json N/A ✅ PASS
Deno oak N/A 1/1 passed ✅ PASS
Deno std N/A 1/1 passed ✅ PASS
.NET hello-world N/A ✅ PASS
.NET json-parse N/A ✅ PASS
Go color 1/1 passed ✅ PASS
Go env 1/1 passed ✅ PASS
Go uuid 1/1 passed ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx passed ✅ PASS
Node.js execa passed ✅ PASS
Node.js p-limit passed ✅ PASS
Rust fd 1/1 passed ✅ PASS
Rust zoxide 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — PASS

Note: Java's default ~/.m2/repository was root-owned and inaccessible to the runner user; builds/tests were run with -Dmaven.repo.local=/tmp/gh-aw/agent/m2repo as a workaround. All other ecosystems ran with default configuration.

Generated by Build Test Suite for #6789 · aut00 · 26.2 AIC · ⊞ 11.3K ·
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

smoke test results:

  • ✅ GitHub MCP data validated
  • ✅ GitHub.com connectivity
  • ✅ File I/O
  • ✅ BYOK inference

Running in direct BYOK mode (COPILOT_PROVIDER_API_KEY + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI (Foundry, o4-mini-aw)

Overall: PASS

🔑 BYOK (AOAI api-key) report filed by Smoke Copilot BYOK AOAI (api-key)
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

Smoke Test: Docker Sbx Validation

@lpcox

Overall: PASS

📰 BREAKING: Report filed by Smoke Docker Sbx
Add label ready-for-aw to run again

@github-actions

Copy link
Copy Markdown
Contributor

@lpcox Smoke test results:
• PRs: Upgrade gh-aw extension to latest pre-release; Validate bounded-query sandbox runtime matrix
• MCP connectivity: ✅
• HTTP: ✅
• File I/O: ✅
• BYOK inference: ✅
Running in direct BYOK mode (AWF_AUTH_TYPE=github-oidc + AWF_AUTH_AZURE_* + COPILOT_PROVIDER_BASE_URL) via api-proxy → Azure OpenAI authenticated via Microsoft Entra
Overall: PASS

🪪 BYOK (AOAI Entra) report filed by Smoke Copilot BYOK AOAI (Entra)
Add label ready-for-aw to run again

@lpcox
lpcox merged commit 670001c into main Jul 31, 2026
137 checks passed
@lpcox
lpcox deleted the copilot/deps-safe-updates-2026-07-31 branch July 31, 2026 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Deps] Safe dependency updates (2026-07-31)

3 participants