Skip to content

Update npm security configuration based on iosdc-pamphlet-template PR #119 - #148

Closed
mitsuharu with Copilot wants to merge 4 commits into
developfrom
copilot/add-npm-security-support
Closed

Update npm security configuration based on iosdc-pamphlet-template PR #119#148
mitsuharu with Copilot wants to merge 4 commits into
developfrom
copilot/add-npm-security-support

Conversation

Copilot AI commented Jan 10, 2026

Copy link
Copy Markdown

Updates the npm security configuration to match the implementation from iosdc-pamphlet-template PR #119, enhancing protection against supply chain attacks.

Changes

Updated .github/workflows/aikidosec-safe-chain.yml

  • Package manager detection: Added automatic detection for npm, yarn, and pnpm using dorny/paths-filter action
  • Multi-lockfile validation: Added check to fail if multiple lockfiles are changed in a single PR
  • safe-chain upgrade: Updated from v1.1.5 to v1.3.2 using curl installation script
  • GitHub Actions updates: Updated to checkout v6.0.1 and setup-node v6.1.0
  • Conditional installation: Added package manager-specific dependency installation steps with error handling

Added Security Documentation to README.md

  • Local environment setup: Instructions for installing @aikidosec/safe-chain locally
  • CI configuration: Explanation of automatic package checks in GitHub Actions
  • Version update guide: Instructions for updating safe-chain version in the workflow

These changes align with the security practices established in the iosdc-pamphlet-template repository, providing comprehensive protection against malicious npm packages while supporting multiple package managers.

Testing

  • ✅ TypeScript compilation passes
  • ✅ Linting passes
  • ✅ All unit tests pass
  • ✅ CodeQL security scan passes with 0 alerts

The changes maintain compatibility with the existing Yarn 4 workflow while adding enhanced security checks and multi-package-manager support.

Original prompt

This section details on the original issue you should resolve

<issue_title>npmのセキュリティ対応を追加する</issue_title>
<issue_description>mitsuharu/iosdc-pamphlet-template#119

修正内容はリンクした先のprを参照してください</issue_description>

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: mitsuharu <296276+mitsuharu@users.noreply.github.com>
Copilot AI changed the title [WIP] Add npm security support Add npm security configuration to prevent supply chain attacks Jan 10, 2026
Copilot AI requested a review from mitsuharu January 10, 2026 00:59

@mitsuharu mitsuharu left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

mitsuharu/iosdc-pamphlet-template#119
このPRのようにしてください

Copilot AI and others added 2 commits January 10, 2026 01:27
…t-template

Co-authored-by: mitsuharu <296276+mitsuharu@users.noreply.github.com>
Co-authored-by: mitsuharu <296276+mitsuharu@users.noreply.github.com>

Copilot AI commented Jan 10, 2026

Copy link
Copy Markdown
Author

mitsuharu/iosdc-pamphlet-template#119
このPRのようにしてください

Updated to match PR #119. Changes include:

  • Enhanced aikidosec-safe-chain workflow with package manager detection and multi-lockfile validation (v1.3.2)
  • Added security documentation to README explaining local setup and CI configuration
  • Removed .npmrc file as it was not part of the referenced PR

Commit: cbdcc14

Copilot AI changed the title Add npm security configuration to prevent supply chain attacks Update npm security configuration based on iosdc-pamphlet-template PR #119 Jan 10, 2026
Copilot AI requested a review from mitsuharu January 10, 2026 01:30
@mitsuharu mitsuharu closed this Jan 10, 2026
@mitsuharu
mitsuharu deleted the copilot/add-npm-security-support branch January 10, 2026 01:30
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.

npmのセキュリティ対応を追加する

2 participants