FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Comparing 2b71d53...02b3cef · SocketDev/socket-cli · GitHub

Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: SocketDev/socket-cli
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 2b71d53
Choose a base ref
Could not load branches
Nothing to show
{{ refName }}
...
head repository: SocketDev/socket-cli
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 02b3cef
Choose a head ref
Could not load branches
Nothing to show
{{ refName }}
  • 3 commits
  • 4 files changed
  • 1 contributor

Commits on Aug 5, 2026

  1. fix(deps): declare form-data so a fresh install can upload

    The bundled Socket SDK loads form-data through a runtime require the
    first time a command uploads files. form-data was only reachable in this
    repo's dev tree, through the pnpm override, so the published package
    shipped a require for something a consumer never installed: a fresh
    install threw "Cannot find module 'form-data'" on the first upload, and
    `socket scan create` in a clean CI job could not get past it.
    
    Declaring it in dependencies puts it in the consumer's install. The
    override stays as is so the dev tree and the published tree agree on
    4.0.6.
    jdalton committed Aug 5, 2026
    Configuration menu
    Copy the full SHA
    8480430 View commit details
    Browse the repository at this point in the history
  2. test(dist): assert every runtime require in dist is declared or builtin

    Parses the built dist/*.js with rollup's parser and collects every
    require()/createRequire() call made with a string literal, then checks
    that each bare specifier names a Node builtin or a package.json
    dependencies entry. That is the contract a consumer's install has to
    satisfy, and nothing else was checking it.
    
    Parsing instead of grepping matters here: vendor.js repeats several of
    those specifiers inside doc comments and code-generation strings, so a
    text scan reports micromatch, picomatch, braces and ajv subpaths as
    findings that are not really there.
    
    Run against the dependencies block before the previous commit, the test
    fails with:
    
      - []
      + [
      +   "dist/vendor.js requires \"form-data\" (package \"form-data\")",
      + ]
    
    The scan stays on dist/. external/ ships copies of blessed and
    blessed-contrib whose terminal widget requires pty.js and term.js behind
    a try/catch, which is a separate question from what the bundle needs.
    jdalton committed Aug 5, 2026
    Configuration menu
    Copy the full SHA
    0c3fea0 View commit details
    Browse the repository at this point in the history
  3. chore(release): 1.1.154

    jdalton committed Aug 5, 2026
    Configuration menu
    Copy the full SHA
    02b3cef View commit details
    Browse the repository at this point in the history
Loading

Back | FazBrowse Home | New Git URL