Skip to content

DX-2866: point agents at start-redis for throwaway databases - #20

Merged
CahidArda merged 3 commits into
mainfrom
DX-2866-start-redis
Jul 31, 2026
Merged

DX-2866: point agents at start-redis for throwaway databases#20
CahidArda merged 3 commits into
mainfrom
DX-2866-start-redis

Conversation

@CahidArda

Copy link
Copy Markdown
Contributor

Mentions https://upstash.com/start-redis in the README quickstart and in the redis_database_create_new tool description, so an agent that needs a scratch Redis for itself doesn't create one in the user's account.

Mentions https://upstash.com/start-redis in the README quickstart and in
the redis_database_create_new tool description, so an agent that needs a
scratch Redis for itself doesn't create one in the user's account.
@linear-code

linear-code Bot commented Jul 27, 2026

Copy link
Copy Markdown

DX-2866

Adds an MCP tool that POSTs to https://upstash.com/start-redis and returns
the markdown response (endpoint, token, claim URL, quickstart). It sends no
API key and creates nothing in the user's account, so it is marked readonly
and stays available when the server runs with a readonly key.

Optional database_id re-fetches an earlier database via the Idempotency-Key
header. The response body carries a live token, so only the status code is
logged. redis_database_create_new now points at this tool for the case where
the agent needs a database for itself.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new Redis tool that lets agents start a temporary Upstash Redis database via https://upstash.com/start-redis (no Upstash account or API key), and updates docs/tool descriptions to steer “scratch DB” use cases toward it instead of creating resources in the user’s account.

Changes:

  • Added redis_database_start_free tool that POSTs to Upstash Start Redis and returns the markdown credentials/quickstart.
  • Registered the new tool in the Redis tool set and verified it’s available under readonly tool filtering.
  • Updated README and redis_database_create_new description to recommend the new tool for throwaway databases.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/tools/redis/start-redis.ts Introduces redis_database_start_free tool that calls Upstash Start Redis and returns markdown credentials.
src/tools/redis/index.ts Registers the new Start Redis tool in the Redis tool registry.
src/tools/redis/db.ts Updates redis_database_create_new description to point agents to redis_database_start_free for scratch DBs.
src/readonly.test.ts Includes startRedisTools in readonly filtering tests and asserts the tool is readonly.
README.md Adds Quickstart tip and example prompt pointing users/agents at Start Redis for temporary DBs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/tools/redis/start-redis.ts
Comment thread src/tools/redis/start-redis.ts Outdated
Review caught a real leak: server.ts logs every tool result text, so the
markdown body returned by redis_database_start_free (which carries a live
token) would reach the logs even though the handler itself avoids logging
it. The redaction in log() only masks JSON-shaped secrets, so it does not
help here.

Adds a `sensitive` flag to CustomTool, marks the tool, and makes the
server log a placeholder instead of the result text.
@CahidArda
CahidArda merged commit e91f5ea into main Jul 31, 2026
2 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.

3 participants