Skip to content

refactor: simplify CLI surface and add end-to-end smoke coverage#53

Closed
ianalloway wants to merge 1 commit into
mainfrom
codex/desloppify-cli-and-e2e
Closed

refactor: simplify CLI surface and add end-to-end smoke coverage#53
ianalloway wants to merge 1 commit into
mainfrom
codex/desloppify-cli-and-e2e

Conversation

@ianalloway

Copy link
Copy Markdown
Owner

What changed

  • Replaced the hand-written if/elif CLI router with a compact command table.
  • Removed the duplicate top-level retry command; solvent jobs retry <id> is now the single retry path.
  • Made jobs retry actually work and normalized --json handling across subcommands.
  • Removed the unsolicited background PyPI thread; upgrade checks are explicit via solvent upgrade.
  • Replaced fragile INSERT OR REPLACE webhook logging with a real upsert that preserves the original receive timestamp.
  • Added a context manager/close() for webhook logging and made replay verification explicit.
  • Deduplicated server job submission and brief lookup helpers.
  • Simplified event capture so server mode no longer double-captures agent events.
  • Tightened Telegram HTTP handling and dependency guidance.
  • Removed the redundant architecture document and collapsed duplicated README/production/Telegram documentation.
  • Added lightweight end-to-end smoke tests for the CLI, offline demo, guardrails, doctor, worker, web app, and optional Telegram path.

Verification encoded in CI

The branch adds an e2e job that runs:

  • CLI help/version/finance/jobs/webhooks
  • offline batch demo
  • guardrail demo
  • doctor (allowing expected missing-live-integration diagnostics)
  • one-pass worker
  • web health/dashboard/chat/job submission/brief authorization flows
  • Telegram application construction with a fake token when the optional extra is installed

Notes

This is deliberately a simplification pass: one command path per operation, fewer background side effects, less duplicate documentation, and broader feature-surface smoke coverage.

@ianalloway

Copy link
Copy Markdown
Owner Author

Closing as superseded by desloppify CLI simplification (#55).

@ianalloway ianalloway closed this Jul 14, 2026
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