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
Copy link
Copy Markdown
Contributor

@aasoni aasoni commented Jun 2, 2026

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 [NS 5] CLI: namespace support for spacetime call and spacetime sql Jun 2, 2026
Comment thread crates/cli/src/api.rs
}

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 from fe0654d to a284b57 Compare June 3, 2026 08:03
@aasoni aasoni force-pushed the alessandro/handle-module-mounts-in-cli branch 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
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