Skip to content

[CU-86b8vxxhv] Add namespace create and update CLI commands#157

Merged
patmagee merged 8 commits intomainfrom
add_namespace_create_update_commands-CU-86b8vxxhv
Mar 13, 2026
Merged

[CU-86b8vxxhv] Add namespace create and update CLI commands#157
patmagee merged 8 commits intomainfrom
add_namespace_create_update_commands-CU-86b8vxxhv

Conversation

@patmagee
Copy link
Contributor

Summary

  • Add namespaces create command with --name (required) and --description (optional) flags
  • Add namespaces update command with positional namespace ID, --name/--description flags, JSON Patch with ETag-based optimistic locking, and 409 conflict handling
  • Add NamespaceCreateRequest model and create_namespace/update_namespace client methods on WorkbenchUserClient
  • Add design spec for the feature

Test plan

  • Unit tests for NamespaceCreateRequest serialization (3 tests)
  • Unit tests for create CLI command (3 tests)
  • Unit tests for update CLI command (6 tests)
  • Full unit test suite passes (375 passed, 0 failures)
  • Lint clean
  • Manual E2E: omics workbench namespaces create --name "Test"
  • Manual E2E: omics workbench namespaces update <id> --name "Updated"
  • Manual E2E: Verify 409 conflict handling with concurrent updates

🤖 Generated with Claude Code

patmagee and others added 7 commits March 13, 2026 14:22
…mmands

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…e commands

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…lict handling

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@platform-automation-dnastack

@github-actions
Copy link

github-actions bot commented Mar 13, 2026

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
10787 6063 56% 30% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
dnastack/cli/commands/workbench/namespaces/commands.py 83% 🟢
dnastack/client/workbench/workbench_user_service/client.py 51% 🟢
dnastack/client/workbench/workbench_user_service/models.py 100% 🟢
TOTAL 78% 🟢

updated for commit: 5807e74 by action🐍

…mespace

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 56.09756% with 18 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@5e4e022). Learn more about missing BASE report.

Files with missing lines Patch % Lines
.../client/workbench/workbench_user_service/client.py 15.00% 17 Missing ⚠️
...tack/cli/commands/workbench/namespaces/commands.py 94.44% 1 Missing ⚠️
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #157   +/-   ##
=======================================
  Coverage        ?   56.20%           
=======================================
  Files           ?      176           
  Lines           ?    10787           
  Branches        ?        0           
=======================================
  Hits            ?     6063           
  Misses          ?     4724           
  Partials        ?        0           
Flag Coverage Δ
unittests 56.20% <56.09%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@mpanik mpanik left a comment

Choose a reason for hiding this comment

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

LGTM

@patmagee patmagee merged commit 7ac9a44 into main Mar 13, 2026
6 checks passed
@patmagee patmagee deleted the add_namespace_create_update_commands-CU-86b8vxxhv branch March 13, 2026 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants