Skip to content

deps: bump js-yaml to 4.3.1 in runtime dependency and lockfile - #6831

Merged
lpcox merged 3 commits into
mainfrom
copilot/deps-safe-dependency-updates
Aug 2, 2026
Merged

deps: bump js-yaml to 4.3.1 in runtime dependency and lockfile#6831
lpcox merged 3 commits into
mainfrom
copilot/deps-safe-dependency-updates

Conversation

Copilot AI commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

This PR applies the safe patch-level dependency update requested by the dependency security monitor: js-yaml from 4.3.0 to 4.3.1. Scope is intentionally limited to dependency metadata and lock resolution, with no feature or behavior changes introduced.

  • Dependency update

    • Updated js-yaml in package.json:
      • dependencies.js-yaml: ^4.3.0^4.3.1
    • Updated override to keep transitive alignment:
      • overrides.markdownlint-cli2.js-yaml: ^4.3.0^4.3.1
  • Lockfile resolution

    • Regenerated package-lock.json so top-level and resolved entries point to js-yaml@4.3.1 with updated integrity metadata.
{
  "dependencies": {
    "js-yaml": "^4.3.1"
  },
  "overrides": {
    "markdownlint-cli2": {
      "js-yaml": "^4.3.1"
    }
  }
}

Copilot AI linked an issue Aug 2, 2026 that may be closed by this pull request
3 tasks
Copilot AI changed the title [WIP] Update js-yaml dependency to version 4.3.1 deps: bump js-yaml to 4.3.1 in runtime dependency and lockfile Aug 2, 2026
Copilot AI requested a review from lpcox August 2, 2026 00:43
@github-actions github-actions Bot mentioned this pull request Aug 2, 2026
@lpcox
lpcox marked this pull request as ready for review August 2, 2026 17:34
Copilot AI review requested due to automatic review settings August 2, 2026 17:34

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 js-yaml from 4.3.0 to 4.3.1 while preserving dependency alignment.

Changes:

  • Bumps the runtime dependency and markdownlint override.
  • Regenerates the lockfile with the matching resolution and integrity hash.
Show a summary per file
File Description
package.json Updates dependency constraints to 4.3.1.
package-lock.json Locks js-yaml to 4.3.1.

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: 0
  • Review effort level: Balanced

@github-actions

github-actions Bot commented Aug 2, 2026

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

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

🌑 The shadows whisper... Smoke Codex failed. The oracle requires further meditation...

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Gemini reports failed. Facets need polishing...

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Claude failed

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Copilot BYOK AOAI (api-key) reports failed. AOAI BYOK (api-key) mode investigation needed...

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

🚀 Security Guard has started processing this pull request

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Chroot tests failed Smoke Chroot failed - See logs for details.

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Copilot BYOK AOAI (Entra) reports failed. AOAI BYOK (Entra) mode investigation needed...

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

🛡️ Smoke Copilot Network Isolation reports failed while checking network isolation. Investigate the egress model.

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Contribution Check failed. Please review the logs for details.

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident...

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Build Test Failed Build Test Suite - See logs for details

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Copilot BYOK reports failed. BYOK mode investigation needed...

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

📡 Smoke OTel Tracing reports failed. OTel tracing regression detected. ⚠️

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Contribution Check completed successfully!

PR #6831 follows the contribution guidelines: it has a clear dependency-update description, references the related issue (#6815), and only changes package metadata/lockfile for the existing dependency. No additional tests or docs updates are needed for this patch-level dependency bump, so no review comment was necessary.

@github-actions

github-actions Bot commented Aug 2, 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 Aug 2, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actions Bot commented Aug 2, 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 Aug 2, 2026

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 98.79% 98.82% 📈 +0.03%
Statements 98.72% 98.74% 📈 +0.02%
Functions 99.02% 99.02% ➡️ +0.00%
Branches 95.11% 95.11% ➡️ +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 Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Test: Copilot BYOK (Direct) Mode ✅

  • ✅ GitHub MCP: PR list verified (merged PRs loaded)
  • ✅ GitHub.com HTTP: 200 OK
  • ✅ File Write/Read: smoke-test-copilot-byok.txt exists and readable
  • ✅ BYOK Inference: Running in direct BYOK mode via api-proxy → api.githubcopilot.com

Result: PASS — Direct BYOK path (COPILOT_PROVIDER_API_KEY) working correctly

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

@github-actions

github-actions Bot commented Aug 2, 2026

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 #6831 · haiku45 · 60.3 AIC · ⊞ 3.6K ·
Add label ready-for-aw to run again

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Test: Copilot Engine

  • ✅ GitHub MCP connectivity (PR list fetch succeeded)
  • ✅ GitHub.com connectivity (HTTP 200)
  • ✅ File write/read test

Recent merged PRs:

Overall: PASS

cc @lpcox

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

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Test Results: Gemini

  • GitHub MCP Testing: ❌
  • GitHub.com Connectivity: ❌
  • File Writing Testing: ✅
  • Bash Tool Testing: ✅

Overall status: FAIL

💎 Faceted by Smoke Gemini
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 Aug 2, 2026
@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Copilot Network Isolation Smoke Test@lpcox

EGRESS_RESULT allow=pass deny=pass

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

Overall: 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 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Test: GitHub Actions Services Connectivity

  • Redis: ❌ (name resolution failure)
  • PostgreSQL pg_isready: ❌ (name resolution failure)
  • PostgreSQL SELECT 1: ❌ (name resolution failure)

Overall: FAIL — host.docker.internal could not be resolved from within the AWF sandbox.

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

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Chroot Version Comparison Results

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

Overall: FAILED — Node.js version differs between host and chroot environments. smoke-chroot label not applied.

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

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor
  • fix: upgrade cli-proxy gh to 2.97.0
  • chore: upgrade gh-aw to v0.84.2 pre-release and recompile workflows
  • GitHub page title check ✅
  • Temp file write/read ✅
  • npm ci && npm run build
  • Overall: PASS

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

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

@lpcox

MCP list call: ✅
GitHub.com connectivity: ✅
Sandbox file I/O: ✅
Inference BYOK mode: ✅

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

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

OTel Tracing Smoke Test Results

  • ✅ Scenario 1 (Module Loading): otel.js loads successfully; isEnabled() → true; exports include startRequestSpan, setTokenAttributes, setBudgetAttributes, endSpan, endSpanError, shutdown, isEnabled, plus internal exporter/serialization helpers.
  • ✅ Scenario 2 (Test Suite): 59/59 tests passed across otel.test.js and otel-fanout.test.js (span creation, gen_ai attributes, budget attributes, OTLP export, fan-out, file exporter).
  • ✅ Scenario 3 (Env Var Forwarding): env-passthrough.ts forwards GITHUB_AW_OTEL_TRACE_ID/GITHUB_AW_OTEL_PARENT_SPAN_ID to the agent; api-proxy-env-config.ts forwards GH_AW_OTLP_ENDPOINTS, OTEL_EXPORTER_OTLP_ENDPOINT, and both trace-context vars to api-proxy.
  • ✅ Scenario 4 (Token Tracker Integration): token-tracker-http.js contains the onUsage callback hook point (4 references) used by the OTEL integration.
  • i️ Scenario 5 (OTEL Diagnostics): No otel.jsonl span file was produced this run (no live LLM API calls occurred to trigger export) — token-usage.jsonl has 9 records from prior tracking. Expected when no OTLP-exportable traffic occurs during the smoke run itself; the pipeline is verified via unit tests.

Overall: ✅ All scenarios pass or are expected/no-op given this run's traffic.

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

@github-actions

github-actions Bot commented Aug 2, 2026

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 ok ✅ PASS
Go env ok ✅ PASS
Go uuid ok ✅ PASS
Java gson 1/1 passed ✅ PASS
Java caffeine 1/1 passed ✅ PASS
Node.js clsx all passed ✅ PASS
Node.js execa all passed ✅ PASS
Node.js p-limit all passed ✅ PASS
Rust fd 1/1 passed ✅ PASS
Rust zoxide 1/1 passed ✅ PASS

Overall: 8/8 ecosystems passed — PASS

Notes:

  • All repositories cloned successfully.
  • Java builds required overriding localRepository to a writable path (/tmp/gh-aw/agent/m2repo) since ~/.m2 was root-owned and not writable in this environment; this is unrelated to the firewall and did not affect proxy configuration testing.

Generated by Build Test Suite for #6831 · auto · 64.5 AIC · ⊞ 11.4K ·
Add label ready-for-aw to run again

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Smoke Test: Docker Sbx Validation

Overall: PASS

cc @lpcox

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

@github-actions

github-actions Bot commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

@lpcox Smoke Test Results:

  • fix: upgrade cli-proxy gh to 2.97.0: ✅
  • chore: upgrade gh-aw to v0.84.2 pre-release and recompile workflows: ✅
  • GitHub MCP Testing: ✅
  • 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 (Foundry, o4-mini-aw)
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 a37b0cf into main Aug 2, 2026
136 of 137 checks passed
@lpcox
lpcox deleted the copilot/deps-safe-dependency-updates branch August 2, 2026 19:46
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 ()

3 participants