Skip to content

feat: added internal database option to project addition - #52

Merged
ymarcon merged 5 commits into
masterfrom
feat/project-internal-db
Sep 9, 2026
Merged

ymarcon merged 5 commits into
masterfrom
feat/project-internal-db

Conversation

@ymarcon

@ymarcon ymarcon commented Sep 9, 2026

Copy link
Copy Markdown
Member

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The create-project payload now always sends internalDatabase: false when no database is provided, which can alter API compatibility/semantics and should be made backward-safe (and covered by tests).

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

This PR adds an “internal database” option when creating Opal projects via the Python client/CLI, and bumps the package version to 6.2.0 alongside a few formatting/doc touch-ups.

Changes:

  • Add --internal/-i option (argparse + Typer CLI) and plumb it into ProjectService.add_project().
  • Include an internalDatabase flag in the project creation payload when no explicit database is provided.
  • Version bump to 6.2.0 plus minor formatting updates (README example, docstring, request chaining).
File summaries
File Description
uv.lock Bumps locked package version to 6.2.0.
README.md Reformats example code (string quoting / line wrapping).
pyproject.toml Bumps project version to 6.2.0.
obiba_opal/system.py Docstring formatting adjustment.
obiba_opal/project.py Adds --internal option and includes internalDatabase in create-project payload.
obiba_opal/file.py Refactors request chaining formatting and simplifies options dict formatting.
obiba_opal/commands.py Adds Typer CLI --internal/-i option and passes it through.
Review details
  • Files reviewed: 5/7 changed files
  • Comments generated: 3
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread obiba_opal/project.py Outdated
Comment thread obiba_opal/project.py
Comment thread obiba_opal/system.py Outdated
ymarcon and others added 2 commits September 9, 2026 21:26
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@ymarcon
ymarcon merged commit 91d444e into master Sep 9, 2026
8 checks passed
@ymarcon
ymarcon deleted the feat/project-internal-db branch September 9, 2026 19:29
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.

2 participants