Skip to content

[NS 5] CLI: namespace support for spacetime call and spacetime sql#5171

Open
aasoni wants to merge 1 commit into
alessandro/typescript/client-codegen-for-mountsfrom
alessandro/handle-module-mounts-in-cli
Open

[NS 5] CLI: namespace support for spacetime call and spacetime sql#5171
aasoni wants to merge 1 commit into
alessandro/typescript/client-codegen-for-mountsfrom
alessandro/handle-module-mounts-in-cli

Conversation

@aasoni

@aasoni aasoni commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Description of Changes

Allow CLI commands to handle calls to namespaced reducers (e.g. spacetime call module lib.library_reducer) and query namespaced talbes (e.g. spacetime sql 'select * from lib.library_table)

API and ABI breaking changes

No

Expected complexity level and risk

2 - Change if fairly small and straightforward.

Testing

Tests performed on a sample module with mounts:

  • CLI can subscribe to lib.library_table
  • CLI can subscribe to lib.library_view
  • CLI can call lib/library_reducer
  • CLI can call lib/library_procedure
  • CLI can subscribe to lib.sublib.sublib_table
  • CLI can subscribe to lib.sublib.sublib_view
  • CLI can call lib/sublib/sublib_reducer
  • CLI can call lib/sublib/sublib_procedure
@aasoni aasoni changed the title CLI: namespace support for spacetime call and spacetime sql Jun 2, 2026
Comment thread crates/cli/src/api.rs Outdated
}

pub async fn call(&self, reducer_name: &str, arg_json: String) -> anyhow::Result<reqwest::Response> {
let encoded = reducer_name.replace('/', "%2F");

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.

Is this supposed to be a . or a /?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think "/" is correct. When calling a reducer or procedure the proposal said the namespace separating character is "/"

@aasoni aasoni force-pushed the alessandro/typescript/client-codegen-for-mounts branch from 6870b0c to f075552 Compare June 3, 2026 08:03
@aasoni aasoni force-pushed the alessandro/handle-module-mounts-in-cli branch 2 times, most recently from a284b57 to e54ed46 Compare June 3, 2026 12:22
@aasoni aasoni force-pushed the alessandro/typescript/client-codegen-for-mounts branch from f075552 to 3b6080d Compare June 3, 2026 12:22
@aasoni aasoni force-pushed the alessandro/typescript/client-codegen-for-mounts branch from 3b6080d to 775649b Compare June 10, 2026 14:00
@aasoni aasoni force-pushed the alessandro/handle-module-mounts-in-cli branch from e54ed46 to 214bc9a Compare June 10, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants