hooks icon indicating copy to clipboard operation
hooks copied to clipboard

feat: add @midwayjs/dev-pack to accelerate local development

Open Lxxyx opened this issue 2 years ago • 2 comments

Ref:

  • https://github.com/midwayjs/hooks/issues/334
  • https://github.com/midwayjs/hooks/issues/300
  • https://github.com/midwayjs/hooks/issues/298
  • https://github.com/midwayjs/hooks/issues/373
  • https://github.com/midwayjs/hooks/issues/374
  • https://github.com/midwayjs/hooks/issues/391

Lxxyx avatar May 11 '22 10:05 Lxxyx

⚠️ No Changeset found

Latest commit: 479c50b3d57eeff980b967aacd8c33afb8cab87d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar May 11 '22 10:05 changeset-bot[bot]

Codecov Report

Base: 89.13% // Head: 81.85% // Decreases project coverage by -7.27% :warning:

Coverage data is based on head (479c50b) compared to base (ab0943f). Patch coverage: 68.50% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #337      +/-   ##
==========================================
- Coverage   89.13%   81.85%   -7.28%     
==========================================
  Files          42       45       +3     
  Lines         773      937     +164     
  Branches      144      166      +22     
==========================================
+ Hits          689      767      +78     
- Misses         84      169      +85     
- Partials        0        1       +1     
Impacted Files Coverage Δ
packages/hooks-core/src/api/index.ts 100.00% <ø> (ø)
packages/hooks-core/src/api/operator/http.ts 100.00% <ø> (ø)
packages/hooks-core/src/api/operator/middleware.ts 100.00% <ø> (ø)
packages/hooks-internal/src/hydrate.ts 33.33% <ø> (ø)
packages/hooks-internal/src/router/file.ts 100.00% <ø> (ø)
packages/hooks-internal/src/util.ts 100.00% <ø> (ø)
packages/hooks-upload/src/index.ts 100.00% <ø> (ø)
packages/hooks/src/api/operator/serverless.ts 46.15% <0.00%> (-3.85%) :arrow_down:
packages/serve/src/index.ts 88.00% <ø> (ø)
packages/test-util/src/index.ts 0.00% <0.00%> (ø)
... and 24 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov-commenter avatar May 12 '22 03:05 codecov-commenter

Socket Security Pull Request Report

Dependency issues detected. If you merge this pull request, you will not be alerted to the instances of these issues again.

📜 Install scripts

Install scripts are run when the package is installed. The majority of malware in npm is hidden in install scripts.

Packages should not be running non-essential scripts during install and there are often solutions to problems people solve with install scripts that can be run at publish time instead.

Package Script field Location
[email protected] (added) postinstall package.json via [email protected]
😵‍💫 Bin script confusion

This package has multiple bin scripts with the same name. This can cause non-deterministic behavior when installing or could be a sign of a supply chain attack

Consider removing one of the conflicting packages. Packages should only export bin scripts with their name

Package Bin script Location
[email protected] (added) js-yaml package.json via [email protected], [email protected],examples/api/package.json via @midwayjs/[email protected],examples/api-bundle/package.json via @midwayjs/[email protected],examples/prisma/package.json via @midwayjs/[email protected], @midwayjs/[email protected],examples/react/package.json via @midwayjs/[email protected], @midwayjs/[email protected],examples/react-faas/package.json via @midwayjs/[email protected],examples/react-with-upload/package.json via @midwayjs/[email protected], @midwayjs/[email protected],examples/vue/package.json via @midwayjs/[email protected], @midwayjs/[email protected],examples/vue-faas/package.json via @midwayjs/[email protected],packages/dev-pack/package.json via @midwayjs/[email protected],packages/dev-pack/src/__tests__/fixtures/api/package.json via @midwayjs/[email protected],packages/dev-pack/src/__tests__/fixtures/api-image/package.json via @midwayjs/[email protected],packages/dev-pack/src/__tests__/fixtures/api-server/package.json via @midwayjs/[email protected],packages/hcc/tests/fixtures/api/package.json via @midwayjs/[email protected], @midwayjs/[email protected],packages/hooks-bundler/package.json via @midwayjs/[email protected], @midwayjs/[email protected], @midwayjs/[email protected],packages/hooks-kit/package.json via @midwayjs/[email protected],packages/hooks-upload/package.json via @midwayjs/[email protected],packages/hooks-upload/tests/fixtures/api/package.json via @midwayjs/[email protected],packages/serve/package.json via @midwayjs/[email protected],packages/serve/tests/fixtures/api/package.json via @midwayjs/[email protected]
Pull request report summary
Issue Status
Install scripts ⚠️ 1 issue
Native code ✅ 0 issues
Bin script confusion ⚠️ 1 issue
Bin script shell injection ✅ 0 issues
Unresolved require ✅ 0 issues
Invalid package.json ✅ 0 issues
HTTP dependency ✅ 0 issues
Git dependency ✅ 0 issues
Non-existent author ✅ 0 issues
Potential typo squat ✅ 0 issues
Known Malware ✅ 0 issues
Telemetry ✅ 0 issues
Protestware/Troll package ✅ 0 issues
Bot Commands

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of package-name@version specifiers. e.g. @SocketSecurity ignore [email protected] [email protected]

Ignoring: @parcel/[email protected], @swc/[email protected], @zkochan/[email protected], [email protected]

⚠️ Please accept the latest app permissions to ensure bot commands work properly. Accept the new permissions here.

Powered by socket.dev

socket-security[bot] avatar Nov 01 '22 08:11 socket-security[bot]

@SocketSecurity ignore @parcel/[email protected] @SocketSecurity ignore @swc/[email protected] @SocketSecurity ignore [email protected] @SocketSecurity ignore @zkochan/[email protected] @SocketSecurity ignore [email protected]

Lxxyx avatar Nov 02 '22 03:11 Lxxyx

GPT summary of 118a633ade38b98a0aece2cbe2ed91884c3bcd46:

Error: couldn't generate summary

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 83836e3a1542b42684b77545c313e7cbc41a8232:

Error: couldn't generate summary

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 702b865521156aaf189760477f74db89f3f807b8:

  • Changed the command for the Node.js job in the github action node.js.yml
  • Added a check to the pre-commit hook to ensure that yarn.lock does not contain registry.npm.alibaba-inc.com pre-commit

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 76b6febcb6debd1b61b951254502f7ef4efb6c93:

  • Added support for Node.js 16 on Windows and macOS node.js.yml

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of aacd3e75656ff53c9fbddc6caca458699565e967:

Error: couldn't generate summary

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of acefadaddab202f36385fb825a2fcd782d6f9f51:

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 1477ac0422004cf6560f0fa77c630cd6e46b41b5:

  • Added MIDWAY_HTTP_PORT environment variable utils.ts

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 9b7c840d928c5077ff4ac0a9035940651b1e656f:

  • Changed describe to describe.skip in the test file cli.test.ts

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 734d9494019910d12d41e8da709b0491b1795755:

Error: couldn't generate summary

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 561ba77b6d72ccb731ac1a4dd25edffadb902605:

Error: couldn't generate summary

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of f1fc7dcfdd206f85837259c80f8c19d4cba397b5:

  • Added exit to the list of signals to handle server.ts
  • Changed the argument of server.close() to reason server.ts
  • Added close and exit events to the IPC share.ts
  • Changed the argument of server.close() to middleware test middleware.test.ts
  • Changed the argument of server.close() to server test server.test.ts
  • Changed the argument of server.close() to restart server.ts

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of ccf1cf38a07b49fbd44db09734e5e750a4fb68f6:

  • Replaced require() with requireWithoutCache() in packages/bundler/src/index.ts and packages/hooks-bundler/src/adapter.ts to ensure latest module content when file changes in dev mode
  • Updated transform() function in packages/bundler/src/index.ts to accept requireWithoutCache() as an argument

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 2ce9c97add27bd057f7faa86407512d7b83707a9:

  • Added consola package to package.json package.json
  • Changed isMatch from an async to a sync function server.test.ts
  • Added IMidwayFaaSApplication import to app.ts app.ts
  • Added ServerState import to middleware.ts middleware.ts
  • Added logger import to share.ts share.ts
  • Added handleStarted and handleError functions to server.ts server.ts
  • Added uncaughtException handler to server.ts server.ts

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 6a332303052b39abb534f7ae94dec8de0a2e9781:

  • Changed the proc.send check from ?. to if share.ts

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 18f6b0bb4cd6383687a2de4a8aedd00359919b98:

  • Changed the way the MidwayFrameworkAdapter is initialized component.ts
  • Fixed an issue with the ApiRouter not correctly handling about paths ApiRouter.test.ts
  • Removed IPCMessage from share.ts [packages/dev-pack/src/share.ts]
  • Changed the way the function container is created container.ts
  • Updated typescript version from 4.7.4 to 4.6.4 yarn.lock

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of f3b2b2b9f3656a3df952c1879bc7a52c480e890a:

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 1c2a1edf0c0cc727efcdf7cbaf4d3c41b65bd4e5:

  • Added a DEBUG environment variable with the value hooks* to the Test Node.js and Build jobs in the node.js.yml Github Action node.js.yml

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 38ec781c87c38da31ac1bfba2bf3ff707b178093:

Error: couldn't generate summary

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of b8ef50620431f4b8e7bb1701f212b1b38afb5304:

Error: couldn't generate summary

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of ed15b2b6d80fee113a03211723824644d85bf5f5:

github-actions[bot] avatar Dec 16 '22 09:12 github-actions[bot]

GPT summary of 6847ca34a22d8984d803313b2433825054c35292:

github-actions[bot] avatar Dec 18 '22 14:12 github-actions[bot]

GPT summary of bd23dd618d53d4de975432712c94d3f7c11f195d:

  • Added a note about installing the Visual C++ Redistributable for Windows when running npx degit intro.md
  • Added a link to the API development guide intro.md

github-actions[bot] avatar Dec 18 '22 14:12 github-actions[bot]

GPT summary of 3ab53ba0f2d9d8dd4bd239a285c7338645d59ea1:

  • Added an const export for internal hooks index.ts

github-actions[bot] avatar Dec 18 '22 14:12 github-actions[bot]

GPT summary of 0b78ed61e6ba5dce44df7075ccd43a9188da8853:

  • Updated lodash.groupby to lodash in package.json package.json
  • Changed import of lodash.groupby to lodash/groupby in src/index.ts index.ts
  • Updated @types/lodash.groupby to @types/lodash in package.json package.json
  • Updated @types/lodash in yarn.lock yarn.lock

github-actions[bot] avatar Dec 18 '22 14:12 github-actions[bot]

GPT summary of 5a6b666ac723958a046aa306924d8b62cb92b6a1:

  • Changed the groupBy import from lodash/groupby to lodash index.ts

github-actions[bot] avatar Dec 18 '22 14:12 github-actions[bot]

GPT summary of 31964e1cb585d449a6dd03d888bad27ae694dc8c:

  • Changed the initialization of the devApis to use the existing router instance if available component.ts
  • Renamed the index-handler to indexHandler in getFunctionId file.ts

github-actions[bot] avatar Dec 18 '22 14:12 github-actions[bot]