systemlink-cli 1.4.4__tar.gz → 1.4.7__tar.gz

{systemlink_cli-1.4.4 → systemlink_cli-1.4.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: systemlink-cli
-Version: 1.4.4
+Version: 1.4.7
 Summary: SystemLink Integrator CLI - cross-platform CLI for SystemLink workflows and templates.
 License-File: LICENSE
 Author: Fred Visser

{systemlink_cli-1.4.4 → systemlink_cli-1.4.7}/pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "systemlink-cli"
-version = "1.4.4"
+version = "1.4.7"
 description = "SystemLink Integrator CLI - cross-platform CLI for SystemLink workflows and templates."
 authors = ["Fred Visser <fred.visser@emerson.com>"]
 packages = [{ include = "slcli" }]

{systemlink_cli-1.4.4 → systemlink_cli-1.4.7}/slcli/_version.py

@@ -1,4 +1,4 @@
 """Version information for slcli."""
 
 # This file is auto-generated. Do not edit manually.
-__version__ = "1.4.4"
+__version__ = "1.4.7"

{systemlink_cli-1.4.4 → systemlink_cli-1.4.7}/slcli/skills/systemlink-webapp/SKILL.md

@@ -30,14 +30,14 @@ Ask before generating any code:
 3. **Starting point** — Fresh Angular project, or do they have existing code?
 4. **Auth context** — Will the app run on the same SystemLink instance it calls (same-origin cookie auth), or does it need an API key for a remote server?
 
-You do NOT need to ask about Angular version or Nimble versions — always use the latest (Angular 19, @ni/nimble-angular latest).
+You do NOT need to ask about Angular version or Nimble versions — always use Angular 20 and the latest compatible `@ni/nimble-angular`.
 
 ---
 
 ## Step 2: Scaffold the Angular project
 
 ```bash
-npx -y @angular/cli@latest new <app-name> --routing --style=scss --skip-git --no-standalone
+npx -y @angular/cli@20 new <app-name> --routing --style=scss --skip-git --no-standalone
 cd <app-name>
 npm install @ni/nimble-angular
 ```
@@ -58,22 +58,23 @@ npm install @ni/systemlink-clients-ts
 
 ### Available services (import paths)
 
-| Service | Import path | `baseUrl` (append to `window.location.origin`) |
-|---------|-------------|------------------------------------------------|
-| Feeds | `@ni/systemlink-clients-ts/feeds` | (none — spec paths already include `/nifeed/v1/`) |
-| Tags | `@ni/systemlink-clients-ts/tags` | `+ '/nitag'` |
-| User / Workspaces | `@ni/systemlink-clients-ts/user` | `+ '/niuser/v1'` |
-| Web Application | `@ni/systemlink-clients-ts/web-application` | `+ '/niapp/v1'` |
-| File Ingestion | `@ni/systemlink-clients-ts/file-ingestion` | `+ '/nifile'` |
-| Test Monitor | `@ni/systemlink-clients-ts/test-monitor` | `+ '/nitest'` |
-| Asset Management | `@ni/systemlink-clients-ts/asset-management` | `+ '/niapm'` |
-| Work Items | `@ni/systemlink-clients-ts/work-item` | `+ '/niworkorder'` |
-| Systems Management | `@ni/systemlink-clients-ts/systems-management` | `+ '/nisysmgmt'` |
-| Notebooks | `@ni/systemlink-clients-ts/notebook` | `+ '/ninotebook'` |
+| Service | Import path | Client `baseUrl` |
+|---------|-------------|------------------|
+| Feeds | `@ni/systemlink-clients-ts/feeds` | `window.location.origin` |
+| Tags | `@ni/systemlink-clients-ts/tags` | `window.location.origin + '/nitag'` |
+| User / Workspaces | `@ni/systemlink-clients-ts/user` | `window.location.origin + '/niuser/v1'` |
+| Web Application | `@ni/systemlink-clients-ts/web-application` | `window.location.origin + '/niapp/v1'` |
+| File Ingestion | `@ni/systemlink-clients-ts/file-ingestion` | `window.location.origin + '/nifile'` |
+| Test Monitor | `@ni/systemlink-clients-ts/test-monitor` | `window.location.origin + '/nitestmonitor'` |
+| Asset Management | `@ni/systemlink-clients-ts/asset-management` | `window.location.origin` |
+| Work Items | `@ni/systemlink-clients-ts/work-item` | `window.location.origin` |
+| Work Orders | `@ni/systemlink-clients-ts/work-order` | `window.location.origin` |
+| Systems Management | `@ni/systemlink-clients-ts/systems-management` | `window.location.origin` |
+| Notebooks | `@ni/systemlink-clients-ts/notebook` | `window.location.origin` |
 
 The client factory for each service lives at `@ni/systemlink-clients-ts/<service>/client`.
 
-> **Base URL gotcha:** Each generated client's path depends on its OpenAPI spec base. For **Feeds**, the spec base is `https://host/` and operation paths already include `/nifeed/v1/...`, so use `baseUrl: window.location.origin` with no suffix. For all other services the spec root matches the service prefix (`https://host/nitag`, `https://host/niuser/v1`, etc.), so set `baseUrl: window.location.origin + '/<prefix>'` as shown above.
+> **Base URL gotcha:** Verify the generated operation URLs, not just the service name. In the published `@ni/systemlink-clients-ts` package, `tags` uses `/v2/...` with `baseUrl: origin + '/nitag'`, `user` uses `/users` and `/workspaces` with `baseUrl: origin + '/niuser/v1'`, `web-application` uses `/webapps/...` with `baseUrl: origin + '/niapp/v1'`, `file-ingestion` uses `/v1/...` with `baseUrl: origin + '/nifile'`, and `test-monitor` uses `/v2/...` with `baseUrl: origin + '/nitestmonitor'`. Services like `feeds`, `asset-management`, `systems-management`, `work-item`, `work-order`, and `notebook` already include `/nifeed`, `/niapm`, `/nisysmgmt`, `/niworkitem`, `/niworkorder`, or `/ninotebook` in each operation path, so those clients should use `baseUrl: window.location.origin`.
 
 > **SDK type mismatch fallback:** If a generated SDK function causes `InputFieldValidationError`, verify the actual request body the server expects with a raw `curl` POST. Sometimes the generated types wrap the body in a `{ request: { ... } }` envelope that the server does not accept (or expect a flat body the type shows as nested). Use direct `fetch` with a manually constructed body as a reliable fallback when the SDK types are wrong.
 
@@ -390,10 +391,18 @@ const { data, error } = await buildClient().post<MyResponse, unknown>({
 Always compute the base URL from `window.location.origin` — never hardcode a hostname:
 
 ```typescript
-const BASE_URL = `${window.location.origin}/nitag`;    // Tags
-const BASE_URL = `${window.location.origin}/nitest`;   // Test Monitor
-const BASE_URL = `${window.location.origin}/niapm`;    // Asset Management
-const BASE_URL = `${window.location.origin}/nifile`;   // File Ingestion
+const tagsBaseUrl = `${window.location.origin}/nitag`;                // tags.js -> /v2/...
+const userBaseUrl = `${window.location.origin}/niuser/v1`;            // user.js -> /users, /workspaces
+const webAppBaseUrl = `${window.location.origin}/niapp/v1`;           // web-application.js -> /webapps/...
+const fileIngestionBaseUrl = `${window.location.origin}/nifile`;      // file-ingestion.js -> /v1/...
+const testMonitorBaseUrl = `${window.location.origin}/nitestmonitor`; // test-monitor.js -> /v2/...
+
+const feedsBaseUrl = window.location.origin;                          // feeds.js -> /nifeed/v1/...
+const assetBaseUrl = window.location.origin;                          // asset-management.js -> /niapm/v1/...
+const systemsBaseUrl = window.location.origin;                        // systems-management.js -> /nisysmgmt/v1/...
+const workItemBaseUrl = window.location.origin;                       // work-item.js -> /niworkitem/v1/...
+const workOrderBaseUrl = window.location.origin;                      // work-order.js -> /niworkorder/v1/...
+const notebookBaseUrl = window.location.origin;                       // notebook.js -> /ninotebook/v1/...
 ```
 
 ### Authentication
@@ -608,7 +617,7 @@ node_modules/.bin/ng build --configuration production --output-path dist/<app-na
 ```
 
 - Do **not** pass `--base-href` — that would re-introduce the `<base>` element
-- Output goes to `dist/<app-name>/browser/` (Angular 19)
+- Output goes to `dist/<app-name>/browser/` (Angular 20)
 
 If you hit budget errors, increase limits in `angular.json`:
 
@@ -647,8 +656,8 @@ Save the returned webapp ID — you'll need it for every subsequent redeploy.
 | CSP `unsafe-inline` error | Beasties injects `onload` in style tags | `inlineCritical: false` in angular.json optimization |
 | App stays light inside dark SystemLink shell | Theme-aware aliases defined on `:root` or embedded app not watching host provider | Define color/shadow aliases on `nimble-theme-provider`; sync `currentTheme` from parent provider |
 | `theme="dark"` is set but colors still look light | Checked the attribute only, not the resolved tokens | Inspect `getComputedStyle(themeProvider).getPropertyValue('--ni-nimble-application-background-color')` in the hosted iframe |
-| CORS / status 0 | `basePath` points to different origin | Set `basePath = window.location.origin + '/service-prefix'` |
-| 404 on API calls | Missing service prefix in base URL | e.g., `/nitag` not just `window.location.origin`; Feeds service paths already include `/nifeed/v1/` so use origin alone |
+| CORS / status 0 | `baseUrl` points to the wrong origin or wrong service root | Match the generated client: for example `test-monitor` uses `${window.location.origin}/nitestmonitor`, while `work-item` uses `window.location.origin` |
+| 404 on API calls | Wrong `baseUrl` for the selected client | Only `tags`, `user`, `web-application`, `file-ingestion`, and `test-monitor` need a prefixed `baseUrl`. `feeds`, `asset-management`, `systems-management`, `work-item`, `work-order`, and `notebook` use bare origin because their operation URLs already include the service prefix |
 | `InputFieldValidationError` on API call | SDK-generated request body has wrong shape | Inspect raw API; the generated type may add or omit a `request: {}` wrapper. Use direct `fetch` with manually constructed body |
 | nimble-dialog does not open | `*ngIf` destroys element before `ViewChild` can resolve | Remove `*ngIf` from the dialog element; use `@ViewChild` + `ElementRef` and call `nativeElement.show()` / `nativeElement.close()` |
 | Icon module import fails | Icon sub-path `@ni/nimble-angular/icons/...` does not exist | Import icon modules from the main `@ni/nimble-angular` barrel only |
@@ -670,18 +679,22 @@ If the host and iframe are same-origin, Playwright or DevTools can inspect `ifra
 
 ---
 
-## Known SystemLink service prefixes
-
-| Service | URL prefix |
-|---------|-----------|
-| Tag Historian | `/nitag/v2` |
-| Test Monitor | `/nitest` |
-| Asset Management | `/niapm` |
-| Systems Management | `/nisysmgmt` |
-| Work Orders | `/niworkorder` |
-| Feeds (Package Manager) | `/nifeed` |
-| Files | `/nifile` |
-| Notebooks | `/ninotebook` |
+## Known SystemLink client base URLs
+
+| Service | Client `baseUrl` |
+|---------|------------------|
+| Tag Historian | `window.location.origin + '/nitaghistorian'` |
+| Tags | `window.location.origin + '/nitag'` |
+| User / Workspaces | `window.location.origin + '/niuser/v1'` |
+| Web Application | `window.location.origin + '/niapp/v1'` |
+| File Ingestion | `window.location.origin + '/nifile'` |
+| Test Monitor | `window.location.origin + '/nitestmonitor'` |
+| Asset Management | `window.location.origin` |
+| Systems Management | `window.location.origin` |
+| Work Items | `window.location.origin` |
+| Work Orders | `window.location.origin` |
+| Feeds (Package Manager) | `window.location.origin` |
+| Notebooks | `window.location.origin` |
 
 See `references/systemlink-services.md` for full API details.
 

{systemlink_cli-1.4.4 → systemlink_cli-1.4.7}/slcli/user_click.py

@@ -13,6 +13,7 @@ from uuid import uuid4
 
 import click
 import questionary
+from click.core import ParameterSource
 
 from .cli_utils import paginate_list_output, validate_output_format
 from .utils import (
@@ -25,6 +26,10 @@ from .utils import (
 from .workspace_utils import get_workspace_display_name, resolve_workspace_id
 
 
+USER_QUERY_PAGE_SIZE = 100
+USER_JSON_DEFAULT_TAKE = 1000
+
+
 def _get_policy_details(policy_id: str) -> Optional[dict]:
     """Fetch policy details from the Auth service.
 
@@ -412,6 +417,7 @@ def _query_all_users(
     sortby: str = "firstName",
     order: str = "asc",
     include_disabled: bool = False,
+    max_items: Optional[int] = None,
 ) -> list:
     """Query all users from the API with server-side pagination using continuation tokens.
 
@@ -430,6 +436,7 @@ def _query_all_users(
         sortby: Field to sort by
         order: Sort order ('asc' or 'desc')
         include_disabled: Whether to include disabled users
+        max_items: Maximum number of users to return. When omitted, fetches all pages.
 
     Returns:
         List of all users
@@ -437,7 +444,7 @@ def _query_all_users(
     url = f"{get_base_url()}/niuser/v1/users/query"
     all_users = []
     continuation_token = None
-    page_size = 100  # API maximum take limit is 100
+    remaining_items = max_items if max_items and max_items > 0 else None
 
     # Build the base filter - combine user filter with active status filter if needed
     combined_filter = filter_str
@@ -451,8 +458,13 @@ def _query_all_users(
             combined_filter = active_filter
 
     while True:
+        request_take = (
+            USER_QUERY_PAGE_SIZE
+            if remaining_items is None
+            else min(USER_QUERY_PAGE_SIZE, remaining_items)
+        )
         payload = {
-            "take": page_size,
+            "take": request_take,
             "sortby": sortby,
             "order": "ascending" if order == "asc" else "descending",
         }
@@ -472,6 +484,11 @@ def _query_all_users(
 
         all_users.extend(users)
 
+        if remaining_items is not None:
+            remaining_items -= len(users)
+            if remaining_items <= 0:
+                return all_users[:max_items]
+
         # Check for continuation token to get next page
         continuation_token = data.get("continuationToken")
         if not continuation_token:
@@ -499,8 +516,10 @@ def register_user_commands(cli: click.Group) -> None:
         "-t",
         type=int,
         default=25,
-        show_default=True,
-        help="Maximum number of users to return",
+        help=(
+            "Table page size (default 25). JSON returns up to 1000 users by default; "
+            "use --take to override."
+        ),
     )
     @click.option(
         "--format",
@@ -573,59 +592,34 @@ def register_user_commands(cli: click.Group) -> None:
             if user_type != "all":
                 type_filter = f'type = "{user_type}"'
 
-            # For JSON format, we can respect the take parameter and use server-side pagination
-            # For table format, we fetch all users and do client-side pagination for better UX
-            if format_output.lower() == "json":
-                # Use server-side pagination for JSON output
-                url = f"{get_base_url()}/niuser/v1/users/query"
-
-                # Build the filter - combine search filter with active status filter if needed
-                combined_filter = search_filter
-                if not include_disabled:
-                    # Add active status filter to the query using correct Dynamic LINQ syntax
-                    # Note: User API uses 'status' field with values 'pending' or 'active'
-                    active_filter = 'status = "active"'
-                    if combined_filter:
-                        combined_filter = f"({combined_filter}) and {active_filter}"
-                    else:
-                        combined_filter = active_filter
-
-                # Add type filter
-                if type_filter:
-                    if combined_filter:
-                        combined_filter = f"({combined_filter}) and {type_filter}"
-                    else:
-                        combined_filter = type_filter
-
-                payload = {
-                    "take": take,
-                    "sortby": sortby,
-                    "order": "ascending" if order == "asc" else "descending",
-                }
-
+            combined_filter = search_filter
+            if type_filter:
                 if combined_filter:
-                    payload["filter"] = combined_filter
+                    combined_filter = f"({combined_filter}) and {type_filter}"
+                else:
+                    combined_filter = type_filter
 
-                resp = make_api_request("POST", url, payload=payload)
-                data = resp.json()
-                users = data.get("users", [])
+            if format_output.lower() == "json":
+                ctx = click.get_current_context()
+                take_source = ctx.get_parameter_source("take")
+                json_take = (
+                    USER_JSON_DEFAULT_TAKE if take_source == ParameterSource.DEFAULT else take
+                )
+
+                users = _query_all_users(
+                    filter_str=combined_filter,
+                    sortby=sortby,
+                    order=order,
+                    include_disabled=include_disabled,
+                    max_items=json_take,
+                )
 
                 click.echo(json.dumps(users, indent=2))
                 return
             else:
                 # For table format, fetch all users for proper client-side pagination
-                # Combine filters for table output
-                combined_filter_for_table = search_filter
-                if type_filter:
-                    if combined_filter_for_table:
-                        combined_filter_for_table = (
-                            f"({combined_filter_for_table}) and {type_filter}"
-                        )
-                    else:
-                        combined_filter_for_table = type_filter
-
                 all_users = _query_all_users(
-                    filter_str=combined_filter_for_table,
+                    filter_str=combined_filter,
                     sortby=sortby,
                     order=order,
                     include_disabled=include_disabled,