primethink-cli 1.3.0__tar.gz → 1.3.2__tar.gz

{primethink_cli-1.3.0/primethink_cli.egg-info → primethink_cli-1.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: primethink-cli
-Version: 1.3.0
+Version: 1.3.2
 Summary: PrimeThink CLI - A powerful tool for interacting with PrimeThink AI API
 Author-email: PrimeThink <support@primethink.ai>
 License-Expression: MIT

primethink_cli-1.3.2/docs/superpowers/specs/2026-08-16-install-attached-skills-design.md

@@ -0,0 +1,184 @@
+# Design: Install skills attached to a chat or agent
+
+**Date:** 2026-08-16
+**Status:** Approved for planning
+
+## Purpose
+
+Add CLI commands that download and install every skill attached to a PrimeThink
+chat and/or agent into a local Claude-Code-style skills directory. The primary
+use case is a sandboxed agent turn, where `PT_CHAT_UUID` / `PT_AGENT_ID` are in
+the environment and a single zero-argument command should fetch everything the
+current chat and agent carry.
+
+## Background: how skills are modeled in the API
+
+Skills are collections, in two flavors:
+
+1. **`skill` collections** — `type == "skill"`. The collection's document tree
+   *is* the skill's file tree; it is downloaded through the same per-collection
+   document endpoints that `pt collections sync-from` uses.
+2. **GitHub-backed skills** — `type == "external_source"` with
+   `external_source_type == "github_skill"`. `external_source_value` holds a
+   **full GitHub URL** pointing at the skill (repo root, or a
+   `/tree/<ref>/<path>` / `/blob/<ref>/<path>` URL naming a subdirectory).
+
+Attachment discovery:
+
+- **Chat:** `GET /api/v1/chats/{chat_id_or_uuid}/collections` — the path
+  parameter accepts an integer ID or a chat UUID. Returns
+  `List[BaseCollectionResponse]` including `type`, `external_source_type`,
+  `external_source_value`, and the link `status`.
+- **Agent:** `GET /api/v1/virtual-assistants/{va_id}/collections` — integer ID
+  only. Returns `BaseCollectionResponse` plus per-agent link fields `hidden`
+  and `status`.
+
+There is no server-side `?type=` filter; filtering is client-side, matching
+what the Deep1 backend itself does internally (`_load_skills`).
+
+Both endpoints authenticate with the standard token, so the CLI's existing
+`_resolve_connection` / `PT_TOKEN` flow works unchanged.
+
+## Commands
+
+Three commands sharing one implementation:
+
+### `pt install-skills` (top-level)
+
+Installs skills from the current chat **and** its agent by default.
+
+- Chat default: `PT_CHAT_UUID` env var, falling back to `PT_CHAT_ID`.
+- Agent default: `PT_AGENT_ID` env var.
+- `--chat-id TEXT` / `--agent-id INT` — explicit overrides.
+- `--chat-only` / `--agent-only` — restrict to one side (mutually exclusive).
+- Standard installer options: `--user` (default, `~/.claude/skills`),
+  `--project` (`./.claude/skills`), `--dir PATH`, `--force`.
+- Standard connection options (`_connection_options`).
+- Errors cleanly (exit 1) if, after env fallback, neither a chat nor an agent
+  can be resolved for the requested sides.
+
+### `pt chat install-skills [CHAT_ID]`
+
+Chat side only. `CHAT_ID` argument (ID or UUID) is optional, falling back to
+`PT_CHAT_UUID` / `PT_CHAT_ID`. Uses `profile_short=False` (the chat group
+reserves `-p` for `--path`). Same installer/connection options as above minus
+the agent-related flags.
+
+### `pt agent install-skills [AGENT_ID]`
+
+Agent side only. `AGENT_ID` (int) optional, falling back to `PT_AGENT_ID`.
+Same installer/connection options minus the chat-related flags.
+
+## Filtering semantics
+
+From each collections listing, keep a collection when:
+
+- `type == "skill"`, **or**
+- `type == "external_source"` and `external_source_type == "github_skill"`.
+
+Additionally:
+
+- **Agent links:** require `status == "active"` and `hidden` falsy — this
+  replicates how Deep1 computes "the skills this agent actually has".
+- **Chat links:** require the link `status` to be `active` when present.
+- When both chat and agent are processed, de-duplicate by collection `id`
+  (first occurrence wins; chat side is processed first).
+
+Non-skill collections (`collection`, `db`, other external-source types) are
+ignored silently.
+
+## Install pipeline
+
+Destination base directory resolves via the existing `_skill_install_base`
+(`--user` / `--project` / `--dir`). Each skill installs into its own
+subdirectory. If the destination subdirectory already exists, the skill is
+**skipped with a notice** unless `--force` is given.
+
+### `skill` collections
+
+- Reuse the existing remote-tree machinery (`_list_remote_tree` and the
+  download helpers behind `pt collections sync-from`) to walk the collection's
+  document tree and download every file into
+  `<skills-dir>/<slug>/`, where `<slug>` is the sanitized collection name
+  (reusing `_sanitize_filename`-style slugging; kebab-case).
+- After download, **warn** (do not fail) if there is no `SKILL.md` at the
+  skill's root.
+
+### `github_skill` collections
+
+- Parse `external_source_value` as a GitHub URL:
+  - `https://github.com/{owner}/{repo}` → repo root, ref `main`, path empty.
+  - `https://github.com/{owner}/{repo}/tree/{ref}/{path...}` (or `/blob/`) →
+    that ref and subdirectory.
+  - Anything else (non-GitHub host, malformed) → per-skill failure.
+- Download via the existing `_download_github_archive` and extract via
+  `_extract_github_subdirectory`, which already enforce SKILL.md presence,
+  path safety (no traversal, no symlinks), and file-count/size caps. Note:
+  the extractor currently rejects an empty source path, so supporting
+  repo-root URLs requires a small extension to it (treat empty path as "the
+  archive's single top-level directory", keeping the SKILL.md marker check).
+- Destination folder name: last path segment of the skill path, or the repo
+  name for root URLs.
+
+## Failure semantics
+
+Consistent with the sync commands: **per-skill failures are non-fatal**. Each
+skill prints an install/skip/fail line as it is processed; the command ends
+with a summary such as `Installed 3, skipped 1, failed 1`.
+
+Exit code 1 when:
+
+- discovery itself fails (the collections request errors), or
+- at least one skill was requested and **all** of them failed.
+
+Exit code 0 otherwise (including "no skills attached", which prints a notice).
+
+## Error handling details
+
+- Unresolvable chat/agent (no arg, no env var): clear message naming the env
+  vars and flags, exit 1.
+- `--chat-only` together with `--agent-only`: usage error.
+- API errors during discovery: standard `_api_request` error path.
+- Download errors, bad GitHub URLs, unsafe archives: recorded as that skill's
+  failure; processing continues.
+
+## Testing
+
+All API traffic is mocked via `primethink.requests.request` (repo convention):
+
+- Discovery: chat and agent collections responses; client-side filter drops
+  non-skill types, inactive links, and hidden agent links.
+- Dedup: same collection attached to both chat and agent installs once.
+- `skill` collection install: mocked directory listing + file downloads;
+  correct tree on disk; SKILL.md warning path.
+- `github_skill` install: URL parsing cases (root, `/tree/`, `/blob/`,
+  malformed); archive extraction reusing the zip-fixture pattern from the
+  developer-skill tests.
+- Env-var defaults (`PT_CHAT_UUID`, `PT_CHAT_ID`, `PT_AGENT_ID`) and explicit
+  overrides; missing-both error.
+- Skip-existing vs `--force`.
+- Failure semantics: partial failure exits 0 with summary; total failure
+  exits 1.
+- All three command entry points share behavior.
+
+## Documentation updates (same change)
+
+- `docs/cli-reference.md` — full reference for all three commands.
+- `skills/primethink-cli/SKILL.md` — command map + sandbox workflow note.
+- `README.md` — commands-at-a-glance table.
+- `USER_GUIDE.md` — walkthrough section.
+- `SPECS.md` — the two collections-listing endpoints.
+- `CHANGELOG.md` — entry under `[Unreleased]`.
+
+**MCP / agent-tools:** deliberately not exposed. Like `pt install-skill`, this
+is a local-filesystem installer, which the MCP server and LangChain plugin do
+not surface.
+
+## Out of scope
+
+- Server-side `?type=` filter on the collections endpoints (five-line backend
+  change; not needed for v1 since client-side filtering is the established
+  pattern).
+- Updating/refreshing already-installed skills beyond `--force` overwrite.
+- Installing skills from collections not attached to a chat/agent (a direct
+  `pt collections install-skill <id>` could be a follow-up).

{primethink_cli-1.3.0 → primethink_cli-1.3.2}/primethink.py

@@ -119,6 +119,17 @@ def _api_call(method, url, token, timeout=REQUEST_TIMEOUT, **kwargs):
     return response, None
 
 
+def _upload_part(path, fileobj):
+    """Multipart tuple for a document upload: (field, (filename, fp, mimetype)).
+
+    Sends the BASENAME (not the local path) and an explicit part
+    Content-Type — requests omits the part header otherwise, which some
+    servers reject (and older PrimeThink APIs answered 500 to).
+    """
+    guessed = mimetypes.guess_type(path)[0] or 'application/octet-stream'
+    return ('documents', (os.path.basename(str(path)), fileobj, guessed))
+
+
 def _api_request(method, url, token, ok_status=(200,), timeout=REQUEST_TIMEOUT, **kwargs):
     """Make an authenticated API request; print an error and exit on failure."""
     response, error = _api_call(method, url, token, timeout=timeout, **kwargs)
@@ -194,7 +205,7 @@ def _upload_file_groups(local_path, file_list, path, upload_batch):
             files_data = []
             for file in dir_files:
                 try:
-                    files_data.append(('documents', stack.enter_context(open(file, 'rb'))))
+                    files_data.append(_upload_part(file, stack.enter_context(open(file, 'rb'))))
                 except OSError as e:
                     error_count += 1
                     click.echo(f"  Error opening {file}: {e}")
@@ -416,7 +427,7 @@ def cli():
 @cli.command()
 def version():
     """Display the version of PrimeThink CLI."""
-    click.echo("PrimeThink CLI v1.3.0")
+    click.echo("PrimeThink CLI v1.3.2")
 
 
 @cli.command()
@@ -802,7 +813,7 @@ def live_app_new(directory, framework, tailwind, flowbite, repo_url, ref):
     )
     click.echo(f"Created PrimeThink Live App at {directory}")
     click.echo(f"Template: {template_id} ({framework}, tailwind={'yes' if tailwind else 'no'}, flowbite={'yes' if flowbite else 'no'})")
-    click.echo("Next: read the generated README.md and rename its sample entity before deploying.")
+    click.echo("Next: read the generated README.md and learn how to build Live Apps.")
 
 
 # ============================================================================
@@ -1324,7 +1335,7 @@ def upload_files(chat_id, files, path, profile, api_url):
 
     params = {'path': path} if path else {}
     with contextlib.ExitStack() as stack:
-        files_data = [('documents', stack.enter_context(open(fp, 'rb'))) for fp in files]
+        files_data = [_upload_part(fp, stack.enter_context(open(fp, 'rb'))) for fp in files]
         response = _api_request(
             'POST', f"{api_url}/api/v1/chats/{chat_id}/documents", token,
             files=files_data, params=params
@@ -1463,7 +1474,7 @@ def upload_files(collection_id, files, path, profile, api_url):
 
     form_data = {'path': path} if path else None
     with contextlib.ExitStack() as stack:
-        files_data = [('documents', stack.enter_context(open(fp, 'rb'))) for fp in files]
+        files_data = [_upload_part(fp, stack.enter_context(open(fp, 'rb'))) for fp in files]
         response = _api_request(
             'POST', f"{api_url}/api/v1/collections/{collection_id}/documents", token,
             files=files_data, data=form_data

{primethink_cli-1.3.0 → primethink_cli-1.3.2/primethink_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: primethink-cli
-Version: 1.3.0
+Version: 1.3.2
 Summary: PrimeThink CLI - A powerful tool for interacting with PrimeThink AI API
 Author-email: PrimeThink <support@primethink.ai>
 License-Expression: MIT

{primethink_cli-1.3.0 → primethink_cli-1.3.2}/primethink_cli.egg-info/SOURCES.txt

@@ -11,6 +11,7 @@ setup.py
 docs/cli-reference.md
 docs/install.md
 docs/primethink_help_llms.txt
+docs/superpowers/specs/2026-08-16-install-attached-skills-design.md
 install/install.cmd
 install/install.ps1
 install/install.sh

{primethink_cli-1.3.0 → primethink_cli-1.3.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "primethink-cli"
-version = "1.3.0"
+version = "1.3.2"
 description = "PrimeThink CLI - A powerful tool for interacting with PrimeThink AI API"
 readme = "README.md"
 license = "MIT"

{primethink_cli-1.3.0 → primethink_cli-1.3.2}/tests/test_cli.py

@@ -1,3 +1,5 @@
+import re
+
 import pytest
 import os
 import json
@@ -51,7 +53,8 @@ def test_version_command(runner):
     """Test version command."""
     result = runner.invoke(cli, ["version"])
     assert result.exit_code == 0
-    assert "PrimeThink CLI v1.3.0" in result.output
+    # Version-agnostic: the hardcoded number went stale at every release.
+    assert re.match(r"PrimeThink CLI v\d+\.\d+\.\d+", result.output)
 
 @patch('primethink.CONFIG_DIR', Path('/tmp/.primethink'))
 @patch('primethink.CONFIG_FILE', Path('/tmp/.primethink/config.json'))
@@ -553,8 +556,10 @@ def test_chat_sync_prefer_local(mock_request, mock_config_fn, runner, tmp_path):
     assert upload_call.kwargs['params'] == {}
     files_sent = upload_call.kwargs['files']
     assert len(files_sent) == 1
-    field_name, file_obj = files_sent[0]
+    field_name, (filename, file_obj, ctype) = files_sent[0]
     assert field_name == 'documents'
+    assert filename == 'shared.txt', "part filename must be the basename, not the local path"
+    assert ctype == 'text/plain', "part must carry an explicit Content-Type"
     assert Path(file_obj.name).name == 'shared.txt'
 
 @patch('primethink.get_active_config')