Skip to content

docs(readme): clarify setup and repository structure - #124

Merged
getdownandcode merged 2 commits into
mainfrom
docs/readme-overview
Nov 2, 2026
Merged

getdownandcode merged 2 commits into
mainfrom
docs/readme-overview

Conversation

@getdownandcode

Copy link
Copy Markdown
Owner

Summary

Reworks the README into a practical guide to understanding and running AgentKit:

  • Explains the run flow, synchronous request behavior, and the roles of Redis and PostgreSQL.
  • Gives a Compose quickstart with the required credentials, a first authenticated API request, and run/trace lookup.
  • Adds a no-service offline demo path.
  • Maps source folders, lists current built-in tools and API routes, and links to deeper documentation.
  • Removes outdated or overly detailed tutorial/provider material and aligns examples with current interfaces.

Validation: reviewed documented commands and endpoints against the current source and Compose configuration; git diff --check passed. Documentation-only change.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-02T18:49:29.058801Z 839fd89 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 839fd897e3

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread README.md
- `calculator` evaluates arithmetic through a restricted AST evaluator.
- `read_file` reads files under the configured sandbox directory and blocks traversal and symlink escapes.
- `http_fetch` fetches HTTP(S) resources while blocking private, loopback, link-local, and metadata addresses.
- `sql_readonly` runs a single read-only query using a dedicated PostgreSQL role. It fails closed when that role is not configured.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Retain the SQL role URL in this tool summary

With test dependencies installed, the default suite will fail at tests/unit/test_docs_accuracy.py::test_readme_does_not_claim_ast_validation_for_sql, which requires this bullet to name DATABASE_READONLY_URL. The rewrite also leaves readers configuring the required fail-closed SQL role without its explicit URL setting; retain that name in the summary.

AGENTS.md reference: AGENTS.md:L115-L117

Useful? React with 👍 / 👎.

@getdownandcode
getdownandcode merged commit 1ce7ffe into main Nov 2, 2026
2 of 3 checks passed
@getdownandcode
getdownandcode deleted the docs/readme-overview branch November 2, 2026 19:02
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