Skip to content

Comments

Add hello world REST endpoint#4

Open
dev-indent[bot] wants to merge 1 commit intomainfrom
indent-2026-02-17-hello-world-endpoint
Open

Add hello world REST endpoint#4
dev-indent[bot] wants to merge 1 commit intomainfrom
indent-2026-02-17-hello-world-endpoint

Conversation

@dev-indent
Copy link

@dev-indent dev-indent bot commented Feb 17, 2026

Summary

  • Adds a minimal Node.js Express server with a GET /hello endpoint returning {"message": "Hello, World!"}
  • Initializes package.json with Express dependency and a start script
  • Server listens on PORT env var (default 3000)

Test plan

  • npm start launches the server
  • curl http://localhost:3000/hello returns {"message":"Hello, World!"}

Generated with Indent - session link

Generated with [Indent](https://indent.com)
Co-Authored-By: Indent <noreply@indent.com>
@dev-indent dev-indent bot added the indent label Feb 17, 2026
@indent
Copy link

indent bot commented Feb 17, 2026

Issues

2 potential issues found:

  • Autofix No .gitignore is included, so a future npm install followed by git add . will commit the entire node_modules directory (hundreds of files) into the repository.
  • Autofix The test script in package.json is the default npm placeholder (echo "Error: no test specified" && exit 1), so npm test will always fail. Consider adding a real test or removing the misleading script.

Summary

This PR adds a minimal Express.js (v5) REST API server with a single GET /hello endpoint to a repository that previously contained only static image assets (logos in .gif, .png, .svg formats).

  • Adds index.js with an Express server listening on PORT (default 3000) serving GET /hello returning { message: "Hello, World!" }
  • Adds package.json with Express ^5.2.1 dependency and a start script
  • Adds package-lock.json lockfile for the dependency tree

CI Checks

Waiting for CI checks...

Autofix All

@@ -0,0 +1,16 @@
{
Copy link

Choose a reason for hiding this comment

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

Missing .gitignore: This PR adds package.json and package-lock.json but no .gitignore. Without one, running npm install and then git add . will commit the entire node_modules directory into the repo. Add a .gitignore with at least node_modules/ to prevent this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants