Fix Licensed CI failure: update .licenses records for @actions/cache v6 and @actions/core v3#4
Conversation
….0.0 Agent-Logs-Url: https://github.com/HiGarfield/cachewrtbuild/sessions/16967279-37e7-4dc2-afeb-141812badc9c Co-authored-by: HiGarfield <32226909+HiGarfield@users.noreply.github.com>
Agent-Logs-Url: https://github.com/HiGarfield/cachewrtbuild/sessions/6400143b-cafb-4248-b8b3-cb09088f9360 Co-authored-by: HiGarfield <32226909+HiGarfield@users.noreply.github.com>
Agent-Logs-Url: https://github.com/HiGarfield/cachewrtbuild/sessions/6400143b-cafb-4248-b8b3-cb09088f9360 Co-authored-by: HiGarfield <32226909+HiGarfield@users.noreply.github.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Pull request overview
Updates Licensed dependency records to match the earlier @actions/* upgrades while making the branch self-consistent with the ESM conversion.
Changes:
- Regenerates
.licensesrecords for@actions/cache@6.0.0and@actions/core@3.0.0to fix Licensed CI failures. - Converts
fetch.jsfrom CommonJSrequireto ESMimportand updates the local import to include the.jsextension.
Reviewed changes
Copilot reviewed 3 out of 5626 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| fetch.js | Switches to ESM imports for GitHub Actions dependencies and local utils. |
| .licenses/npm/@actions/core.dep.yml | Updates Licensed record to @actions/core@3.0.0. |
| .licenses/npm/@actions/cache.dep.yml | Updates Licensed record to @actions/cache@6.0.0. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| import * as core from "@actions/core"; | ||
| import { execSync } from "child_process"; | ||
| import * as cache from "@actions/cache"; | ||
| import { parseBooleanInput, buildBaseConfig } from "./utils.js"; |
There was a problem hiding this comment.
These ESM import statements will throw a syntax/runtime error if fetch.js is still executed as CommonJS (e.g., no \"type\": \"module\" in package.json, or if the runtime expects CJS entrypoints). Ensure the action/package runtime explicitly runs this file as ESM (e.g., set \"type\": \"module\", rename to .mjs and update references, or add a build step that outputs CJS).
| import * as core from "@actions/core"; | |
| import { execSync } from "child_process"; | |
| import * as cache from "@actions/cache"; | |
| import { parseBooleanInput, buildBaseConfig } from "./utils.js"; | |
| const core = require("@actions/core"); | |
| const { execSync } = require("child_process"); | |
| const cache = require("@actions/cache"); | |
| const { parseBooleanInput, buildBaseConfig } = require("./utils.js"); |
PR #3 upgraded
@actions/cache(v4→v6) and@actions/core(v1→v3) with an ESM conversion, but omitted regenerating the.licensesdependency records — causing the Licensed CI check to fail with "cached dependency record out of date".Changes
.licenses/npm/@actions/cache.dep.yml: version bumped4.0.3→6.0.0(regenerated vialicensed cache).licenses/npm/@actions/core.dep.yml: version bumped1.11.1→3.0.0(regenerated vialicensed cache)