Skip to content

Conversation

@Addausi
Copy link

@Addausi Addausi commented Dec 30, 2025

Description

Issue(s) fixed

Fixes #

Preview

Checklist

  • If this PR updates or adds documentation content that changes or adds technical meaning, it has received an approval from an engineer or DevRel from the relevant team.
  • If this PR updates or adds documentation content, it has received an approval from a technical writer.

External contributor checklist

  • I've read the contribution guidelines.
  • I've created a new issue (or assigned myself to an existing issue) describing what this PR addresses.

Note

Adds CI and security metadata.

  • CI: New workflows npm-grunt.yml (Grunt build on Node 18/20/22 for pushes/PRs), npm-publish-github-packages.yml (test on release then publish to GitHub Packages), and trivy-cache.yml (scheduled/PR Trivy cache update)
  • Docs: Adds SECURITY.md with supported versions and vulnerability reporting guidance

Written by Cursor Bugbot for commit 5ce0cf1. This will update automatically on new commits. Configure here.

@Addausi Addausi requested review from a team as code owners December 30, 2025 04:47
@vercel
Copy link

vercel bot commented Dec 30, 2025

@Addausi is attempting to deploy a commit to the Consensys Team on Vercel.

A member of the Team first needs to authorize it.

mvn-toolchain-id: # optional
# Name of Maven Toolchain Vendor if the default name of "${distribution}" is not wanted. See examples of supported syntax in Advanced Usage file
mvn-toolchain-vendor: # optional

Copy link

Choose a reason for hiding this comment

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

Malformed YAML breaks workflow with invalid syntax

The trivy-cache.yml workflow file has been corrupted by accidentally pasting the actions/setup-java action template at the top of the file. Line 1 contains invalid YAML syntax (--- - name: Setup Java JDK with content after the document separator on the same line). The Java setup configuration (lines 1-43) exists outside of any workflow structure and appears to be template/snippet text that was accidentally inserted. The actual workflow definition doesn't begin until line 45. This will cause the GitHub Actions workflow to fail with a YAML parsing error.

Fix in Cursor Fix in Web

- name: Build
run: |
npm install
grunt
Copy link

Choose a reason for hiding this comment

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

Workflow references grunt which isn't a dependency

This workflow runs grunt after npm install, but grunt is not listed as a dependency in package.json. The project uses Docusaurus for building, not Grunt. This workflow will fail on every run because the grunt command won't be found. This appears to be a template workflow that was added without being adapted to this project's actual build system.

Fix in Cursor Fix in Web

- run: npm ci
- run: npm publish
env:
NODE_AUTH_TOKEN: ${{secrets.GITHUB_TOKEN}}
Copy link

Choose a reason for hiding this comment

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

Publish workflow fails on private package

This workflow attempts to run npm publish when a release is created, but package.json has "private": true set. Publishing a private package to npm or GitHub Packages will fail with an error. This workflow appears to be a default template that was added without being configured for this project's actual publishing needs.

Fix in Cursor Fix in Web

with:
node-version: 20
- run: npm ci
- run: npm test
Copy link

Choose a reason for hiding this comment

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

Workflow runs npm test but no test script exists

The build job runs npm test but package.json does not define a test script. This will cause the build job to fail with "Missing script: test" error on every release, preventing the entire publish workflow from completing. The project's scripts include lint, typecheck, and build, but no test command.

Fix in Cursor Fix in Web

@Addausi
Copy link
Author

Addausi commented Dec 30, 2025

Description

Issue(s) fixed

Fixes #

Preview

Checklist

  • If this PR updates or adds documentation content that changes or adds technical meaning, it has received an approval from an engineer or DevRel from the relevant team.
  • If this PR updates or adds documentation content, it has received an approval from a technical writer.

External contributor checklist

  • I've read the contribution guidelines.
  • I've created a new issue (or assigned myself to an existing issue) describing what this PR addresses.

Note

Adds CI and security metadata.

  • CI: New workflows npm-grunt.yml (Grunt build on Node 18/20/22 for pushes/PRs), npm-publish-github-packages.yml (test on release then publish to GitHub Packages), and trivy-cache.yml (scheduled/PR Trivy cache update)
  • Docs: Adds SECURITY.md with supported versions and vulnerability reporting guidance

Written by Cursor Bugbot for commit 5ce0cf1. This will update automatically on new commits. Configure here.

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.

1 participant