Skip to content
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: netlify/primitives
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: edge-functions-v2.16.1
Choose a base ref
...
head repository: netlify/primitives
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: edge-functions-v2.16.2
Choose a head ref
  • 15 commits
  • 48 files changed
  • 8 contributors

Commits on Jul 21, 2025

  1. build: remove nuxt playground typechecking (#367)

    There's some intermittent issue in CI that I can't quite figure out:
    
    ```
    > vue-tsc --noEmit && nuxi prepare playground && cd playground && vue-tsc --noEmit
    
    [success] [nuxi] Types generated in playground/.nuxt
    Error: netlify/edge-functions/geo.ts(1,15): error TS2305: Module '"@netlify/edge-functions"' has no exported member 'Context'.
    Error: netlify/edge-functions/world-adder.ts(1,15): error TS2305: Module '"@netlify/edge-functions"' has no exported member 'Context'.
    ```
    
    (example run: https://github.com/netlify/primitives/actions/runs/16378826884/job/46285296127?pr=365)
    
    I think it's some issue at the intersection of npm workspaces + a non-workspace `package.json`
    within a workspace + referencing a dependency that happens to be in this workspace + maybe CI npm
    caching...?
    
    Doesn't seem worth spending more time on to just type-check the nuxt module playground.
    
    Co-authored-by: Ivan Zarea <zarea.ivan@gmail.com>
    Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>
    3 people authored Jul 21, 2025
    Configuration menu
    Copy the full SHA
    958bab3 View commit details
    Browse the repository at this point in the history
  2. fix(deps): update dependency @netlify/zip-it-and-ship-it to v14 (#334)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
    renovate[bot] authored Jul 21, 2025
    Configuration menu
    Copy the full SHA
    fe935fe View commit details
    Browse the repository at this point in the history
  3. chore(deps): update dependency @netlify/edge-functions to ^2.16.1 (#371)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
    renovate[bot] authored Jul 21, 2025
    Configuration menu
    Copy the full SHA
    55b3a4d View commit details
    Browse the repository at this point in the history
  4. chore: release main (#372)

    Co-authored-by: token-generator-app[bot] <82042599+token-generator-app[bot]@users.noreply.github.com>
    Co-authored-by: Michal Piechowiak <misiek.piechowiak@gmail.com>
    token-generator-app[bot] and pieh authored Jul 21, 2025
    Configuration menu
    Copy the full SHA
    23653db View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    578e5f2 View commit details
    Browse the repository at this point in the history
  6. Configuration menu
    Copy the full SHA
    77e828c View commit details
    Browse the repository at this point in the history

Commits on Jul 22, 2025

  1. chore: fix @netlify/nuxt flakes due to HMR port conflict (#376)

    The nuxt module tests run the module in dev mode (because that's the point of our nuxt module),
    which results in HMR being enabled. Unlike the main vite port, there's no dynamic port selection
    for the HMR WebSocket port, so when the test files run in parallel they often conflict by trying to
    bind to the same port.
    
    I could harcode two different ports with `vite.server.hmr.port`, but there's very little value to
    exercising the HMR code path with these tests, so I just disabled it.
    
    See also very similar withastro/astro#11744.
    serhalp authored Jul 22, 2025
    Configuration menu
    Copy the full SHA
    4386ac8 View commit details
    Browse the repository at this point in the history
  2. chore: fix outstanding eslint errors (#377)

    * refactor: work around strange eslint rule bug
    
    I did a bunch of research on this and I'm fairly confident it's a bug, but I couldn't come up with a
    minimal repro to open an issue and felt like I couldn't justify spending more time here.
    
    This is a valuable rule, so I'd rather not suppress it here or globally.
    
    * refactor: remove unused eslint suppressions
    
    ---------
    
    Co-authored-by: Eduardo Bouças <mail@eduardoboucas.com>
    serhalp and eduardoboucas authored Jul 22, 2025
    Configuration menu
    Copy the full SHA
    f6eec1d View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    c30ac6d View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    a186360 View commit details
    Browse the repository at this point in the history
  5. fix: handle 304 responses from functions (#373)

    * test: add test for function returning 304 response
    
    * test: run tests in dev dir
    
    * fix: maintain bodyless responses
    
    * test: enable watch mode in tests that test modifying functions
    
    * chore: update CODEOWNERS to match all nested paths (#378)
    
    * refactor: prefer early bails, use Set for more performant lookups
    pieh authored Jul 22, 2025
    Configuration menu
    Copy the full SHA
    1094683 View commit details
    Browse the repository at this point in the history
  6. chore: release main (#375)

    Co-authored-by: token-generator-app[bot] <82042599+token-generator-app[bot]@users.noreply.github.com>
    token-generator-app[bot] authored Jul 22, 2025
    Configuration menu
    Copy the full SHA
    ead53c7 View commit details
    Browse the repository at this point in the history
  7. feat: add actual dev geolocation to functions and edge functions cont…

    …ext (#345)
    
    * Initial plan
    
    * Add comprehensive geolocation functionality with API, caching, and mode support
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * Address code review feedback: use LocalState class, remove CLI references, hook up to edge functions
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * Address code review feedback: fix import extension, use proper MockFetch pattern, fix type errors
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * Fix nondeterministic tests by using mock geolocation in test environment
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * Add geolocation.mode option to edge functions config and update tests
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * chore: reformat
    
    * refactor: fix wrong test types
    
    * build(eslint): disable annoying rule in test
    
    * Move geolocation config to top level to apply to both functions and edge functions
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * Refactor geolocation API from mode to enabled/cache booleans
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * Refactor geolocation config to follow existing pattern and simplify API
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * Fix vite plugin integration tests for middleware features message
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * refactor: share geolocation instance across functions and edge functions
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * Remove geoCountry parameter and simplify geolocation API
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    * feat: use Geolocation type for geolocation variable
    
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    
    ---------
    
    Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
    Co-authored-by: serhalp <1377702+serhalp@users.noreply.github.com>
    Co-authored-by: Philippe Serhal <philippe.serhal@netlify.com>
    3 people authored Jul 22, 2025
    Configuration menu
    Copy the full SHA
    916c38b View commit details
    Browse the repository at this point in the history

Commits on Jul 23, 2025

  1. fix(deps): update dependency ipx to v3.1.0 (#381)

    Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
    renovate[bot] authored Jul 23, 2025
    Configuration menu
    Copy the full SHA
    a224a58 View commit details
    Browse the repository at this point in the history
  2. chore: release main (#380)

    Co-authored-by: token-generator-app[bot] <82042599+token-generator-app[bot]@users.noreply.github.com>
    token-generator-app[bot] authored Jul 23, 2025
    Configuration menu
    Copy the full SHA
    513df96 View commit details
    Browse the repository at this point in the history
Loading