Skip to content

server: update port and add smoke tests#266

Draft
osortega wants to merge 2 commits into
masterfrom
copilot-worktree-2026-03-25T02-15-26
Draft

server: update port and add smoke tests#266
osortega wants to merge 2 commits into
masterfrom
copilot-worktree-2026-03-25T02-15-26

Conversation

@osortega
Copy link
Copy Markdown
Owner

Summary

  • keep the server configured on port 1245
  • make index.js export the Express app and port while only listening when run directly
  • replace the placeholder failing npm test script with lightweight Node smoke tests

Why

  • the session change updates the app to the requested port
  • the repository previously had no meaningful passing validation because npm test intentionally failed
  • exporting the app enables simple request-level tests without changing runtime behavior

Validation

  • npm test

Notes for reviewers

  • this PR includes the earlier port-change commit already present on the branch plus a follow-up commit that makes repository validation real and passing

osortega and others added 2 commits March 24, 2026 19:15
Add a lightweight Node test suite so the repository's npm test command validates the home page port output and PATCH route behavior.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant