Skip to content

[codex] Simplify user flow and docs#6

Merged
justAbdulaziz10 merged 1 commit into
mainfrom
codex/simplify-v11
May 6, 2026
Merged

[codex] Simplify user flow and docs#6
justAbdulaziz10 merged 1 commit into
mainfrom
codex/simplify-v11

Conversation

@justAbdulaziz10

Copy link
Copy Markdown
Member

What changed

  • Added simpler commands: report, audit, redact, demo, wizard, and agent-prompt.
  • Added timestamped report folders, --open, --shareable, better missing-path help, and clearer terminal summaries.
  • Made demo generation self-contained in the CLI so installed users can run it.
  • Simplified README, Quickstart, Export Guide, How It Works, and PyPI docs.
  • Added a script and GitHub workflow to regenerate docs/assets/demo-report.png after pushes to main.

Validation

  • make check
  • make demo
  • make path-smoke
  • bash scripts/generate_demo_screenshot.sh
  • make package PYTHON=python3

@justAbdulaziz10 justAbdulaziz10 merged commit 65ee5f0 into main May 6, 2026
12 checks passed
@justAbdulaziz10 justAbdulaziz10 deleted the codex/simplify-v11 branch May 6, 2026 14:03

@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: 99d3ddd8fc

ℹ️ 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 on lines +2247 to +2248
if raw_args and raw_args[0] not in legacy_commands and not raw_args[0].startswith("-"):
raw_args = ["report", *raw_args]

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 Restore legacy invocation when options precede input path

The new legacy shim only prepends report when the first token is not an option, so previously valid calls like inbox-report --no-pdf /path/to/Mail.mbox now fail with an invalid command error instead of running a strict report. This is a behavior regression introduced by the subcommand migration and will break existing scripts/automation that place flags before the positional input (which argparse accepted in the prior CLI).

Useful? React with 👍 / 👎.

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