Skip to content

Remove README content for retired NASA APIs and missing scripts - #14

Merged
ProgramComputer merged 2 commits into
mainfrom
programcomputer/wonderful-wozniak-jkxcuv
Sep 30, 2026
Merged

ProgramComputer merged 2 commits into
mainfrom
programcomputer/wonderful-wozniak-jkxcuv

Conversation

@ProgramComputer

@ProgramComputer ProgramComputer commented Sep 30, 2026 •

Copy link
Copy Markdown
Owner

Checked live on 2026-09-30 before removing:

  • Mars Rover Photos: api.nasa.gov/mars-photos returns HTTP 404 "No such app" with both a personal key and DEMO_KEY, while NeoWs on the same host returns 200. It was removed from the api.nasa.gov catalog (nasa/api-docs) on 2025-10-24 and 2026-03-26. Dropped from the API list, the API Methods and client examples, the sample log line and the .env.example comment.
  • NASA Sounds: api.nasa.gov/planetary/sounds returns HTTP 404. It was retired from the catalog on 2018-06-29. Dropped from the API list.
  • InSight weather and TLE still respond and are still in the catalog, so they stay listed.

Also:

  • scripts/test-with-inspector.sh was never committed, so the Inspector section now runs the official MCP Inspector (npx @modelcontextprotocol/inspector node dist/index.js). Verified: the web UI starts, and a tool call through the Inspector CLI returned live data with the key read from .env.
  • npm run dev doesn't exist; the Development section now uses npm start.
  • The stdio transport note no longer names a specific desktop client.

npm test (118/118), npm run lint and npm run docs:check pass.

Checked live on 2026-09-30 before removing:
- Mars Rover Photos: api.nasa.gov/mars-photos returns HTTP 404 "No such
  app" (also through nasa_mars_rover), while NeoWs on the same host and
  key returns 200. Dropped from the API list, the API Methods and client
  examples, the sample log line and the .env.example comment.
- NASA Sounds: api.nasa.gov/planetary/sounds returns HTTP 404. Dropped
  from the API list.

Also replaced two commands that do not exist: scripts/test-with-inspector.sh
was never committed, so the Inspector section now runs the official MCP
Inspector against dist/index.js, and npm run dev becomes npm start.

InSight weather and TLE still respond upstream, so they stay listed.
@ProgramComputer
ProgramComputer merged commit 7b89699 into main Sep 30, 2026
6 checks passed
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