Skip to content

Update deps and clean quickstart drift#236

Closed
dex-the-ai wants to merge 1 commit into
masterfrom
deps/update-2026-04-29
Closed

Update deps and clean quickstart drift#236
dex-the-ai wants to merge 1 commit into
masterfrom
deps/update-2026-04-29

Conversation

@dex-the-ai
Copy link
Copy Markdown
Contributor

Summary

  • Update the Node quickstart dependencies from current master and refresh the lockfile.
  • Remove stale setup scripts and unused env example fields that no longer match the repo.
  • Clarify the README local setup snippet so it explicitly starts from config/dev.env.example and includes DB_BUCKET_NAME.

Verification

  • fnm use 20 && npm install
  • npm test ✅ (19/19)
  • npm start + manual API walkthrough on localhost ✅

Evidence

  • Fresh rerun from master passed the Jest suite on Node 20.

  • Walkthrough exercised airport list/direct-connections, airline list/to-airport, and full airport/airline/route CRUD against the running app.

  • README/env drift fixed: the repo no longer advertises IS_CAPELLA / DELAY, and the setup snippet now tells readers to copy config/dev.env.example before editing.

  • `

    Walkthrough result summary

    • airport/list → 200 (Abbeville sample)
    • airport/direct-connections → 200 (AAE sample destination)
    • airline/list → 200 (40-Mile Air sample)
    • airline/to-airport → 200 (Air France sample)
    • airport CRUD → 201/200/200/204
    • airline CRUD → 201/200/200/204
    • route CRUD → 201/200/200/204
    `
  • tutorial-maintenance/runs/couchbase-examples__nodejs-quickstart/2026-04-29-fresh-from-main/verification.md

  • tutorial-maintenance/runs/couchbase-examples__nodejs-quickstart/2026-04-29-fresh-from-main/commands.txt

Notes

  • The old init-db / init-test-db / init-test-index scripts still referenced missing files on master, so this PR removes those dead entry points instead of leaving them advertised.
  • I kept the evidence text-first for this fresh rerun; richer Swagger media is possible for this repo, but the log/request proof here already covers the user-facing flow clearly.

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates project dependencies, removes unused packages, and simplifies the configuration by removing several database initialization scripts. The README has been updated with instructions for setting up the local environment. A suggestion was made to include the setup command for the test environment file to ensure that local integration tests can be executed without missing configuration errors.

Comment thread README.md
All configuration for communication with the database is read from the environment variables. We have provided a convenience feature in this quickstart to read the environment variables from a local file, `dev.env` in the `config` folder.

```sh
cp config/dev.env.example config/dev.env
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The npm test script in package.json is configured to use ./config/test.env. Since you are now explicitly instructing users to create config/dev.env from the example file, you should also include the command to create config/test.env. This ensures that users can run the integration tests locally without encountering errors about missing environment files.

Suggested change
cp config/dev.env.example config/dev.env
cp config/dev.env.example config/dev.env
cp config/test.env.example config/test.env

@ejscribner ejscribner closed this Apr 30, 2026
@ejscribner ejscribner deleted the deps/update-2026-04-29 branch April 30, 2026 00:27
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.

2 participants