Skip to content

Conversation

@ritoban23
Copy link

Overview

Implements issue #132 by adding a minimal CLI with three essential commands.

Changes

  • memori --version: Display package version
  • memori init [--force]: Create starter memori.json config
  • memori health [--config <path>] [--check-db]: Validate environment and configuration
  • Console script automatically available after pip install memorisdk
  • 26 comprehensive unit tests (all passing)
  • Complete CLI documentation and troubleshooting guide

Testing

  • All 26 tests pass ✅
  • Manual testing verified all commands work correctly
  • CI/CD ready with proper exit codes

Files Changed

  • memori/cli.py (new) - Main CLI implementation
  • tests/unit/test_cli.py (new) - Comprehensive tests
  • docs/getting-started/cli-reference.md (new) - CLI documentation
  • pyproject.toml (modified) - Console script registration
  • CHANGELOG.md (modified) - Feature entry
  • docs/getting-started/quick-start.md (modified) - CLI mention
  • mkdocs.yml (modified) - Navigation update

Closes #132

…nd health checks; add documentation and tests.
@ritoban23 ritoban23 force-pushed the feature/cli-commands-132 branch from f0c7e67 to af65ab0 Compare October 29, 2025 07:54
@Boburmirzo
Copy link
Contributor

@ritoban23 Thanks for the contribution! We are looking at the PR.

@devwdave
Copy link
Collaborator

devwdave commented Dec 4, 2025

@ritoban23 Thank you for this contribution. The MemoriLabs team recently released version 3 of memori and as a result this PR is no longer applicable. Version 3 has a CLI with a few commands, including the ones listed above. With that said, we'd love to continue to improve the CLI so I encourage you to take a look at v3 and see what you think.

We're regularly reviewing PRs now. I'm eager to see your next one come in!

@devwdave devwdave closed this Dec 4, 2025
@ritoban23
Copy link
Author

@devwdave Sure I'll definitely take a look and try to suggest improvements...also I submitted this PR during hacktoberfest so just curious if I will be eligible for swag?

@devwdave
Copy link
Collaborator

devwdave commented Dec 5, 2025

I'll look into this for you.

@Boburmirzo
Copy link
Contributor

Boburmirzo commented Dec 5, 2025

@devwdave Sure I'll definitely take a look and try to suggest improvements...also I submitted this PR during hacktoberfest so just curious if I will be eligible for swag?

@ritoban23 Thanks for the contribution!

let me work on this. Usually, swag is given for PRs that are successfully merged. Could you share your email and I can follow up with you?

@ritoban23
Copy link
Author

ritoban23 commented Dec 5, 2025

@Boburmirzo thanks I appreciate you looking into it, would be gald to have some cool memori swag.....meanwhile i can look to submit more PRs that get merged!
Here's the email: [email protected]

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.

Add a minimal CLI for memori: init, --version, and health commands

3 participants