Skip to content

Expand setup docs and add runnable examples#12

Merged
aandersen2323 merged 1 commit into
mainfrom
codex/add-detailed-documentation-with-examples
Oct 24, 2025
Merged

Expand setup docs and add runnable examples#12
aandersen2323 merged 1 commit into
mainfrom
codex/add-detailed-documentation-with-examples

Conversation

@aandersen2323
Copy link
Copy Markdown
Owner

Summary

  • expand the README with a step-by-step setup guide, environment configuration notes, and detailed client walkthroughs
  • document CLI usage and add references to new runnable examples for API, MCP, and n8n integrations
  • add runnable example assets (notebook, Python script, n8n workflow) plus CLI requirements to support the docs

Testing

  • not run (documentation-only changes)

https://chatgpt.com/codex/tasks/task_e_68fafb80ce4483318a3cae4ca071aa60

@aandersen2323 aandersen2323 merged commit f6f78ea into main Oct 24, 2025
1 check failed
@aandersen2323 aandersen2323 deleted the codex/add-detailed-documentation-with-examples branch October 24, 2025 04:12
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

FlowDex/README.md

Lines 140 to 144 in 9f07d9a

```bash
curl -X POST http://localhost:8787/memory/put \
-H "Content-Type: application/json" \
-H "x-flowdex-api-key: $FLOWDEX_API_KEY" \
-d '{

P1 Badge Fix authentication header name in curl examples

The new setup walkthrough shows sample requests sending x-flowdex-api-key, but the FastAPI dependency require_api_key reads the X-API-Key header (x_api_key parameter). Anyone who copies these examples while setting FLOWDEX_API_KEY will always receive 401 responses because the API never sees the provided key. The header name in the docs should match what the server actually checks.


"inputs": {
"user": "Draft the customer update for incident INC-2045.",
"context": ["runbook.postgres"],
"tool_hints": ["post_incident_report"],

P1 Badge Update example request schema to match InferRequest

The runnable examples build the request body with an inputs object containing user, context, and tool_hints, but the /infer endpoint expects these fields at the top level (user_input, context_ids, tool_candidates). Pydantic will drop the nested data, so the server receives empty strings and cannot apply the provided context or tool hints. The sample payload should use the actual field names or the script will return meaningless results. The README and notebook examples use the same incorrect structure.

ℹ️ 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".

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.

1 participant