stratyfy-cli 0.2.0__tar.gz → 0.2.1__tar.gz

{stratyfy_cli-0.2.0 → stratyfy_cli-0.2.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: stratyfy-cli
-Version: 0.2.0
+Version: 0.2.1
 Summary: The pre command: describe a database, train a PRE model, read its rules
 Requires-Python: <4,>=3.11
 Requires-Dist: attrs>=23
@@ -44,7 +44,7 @@ failure the command exits with 1 and prints `code`, `message` and `next_step`.
 | `pre guide` | Prints the method a coding agent follows, from set-up to the report. It is the plugin's `onboard` skill, bundled in the CLI. |
 | `pre init --api-key <key> [--api-url <url>]` | Writes a key you already have to `.pre/config.json`. The key can also come from `PRE_API_KEY`. |
 | `pre connect <db>` | Describes a SQLite or Postgres database: tables, row counts, columns, types, null share, sample values and foreign keys. It opens the database read-only. |
-| `pre setup claude [--yes] [--dry-run] [--local]` | For Claude Code. Run it in your own terminal. It adds the API host and a remote Postgres host to Claude Code's sandbox network allowlist. It shows the change and asks first. It refuses to write when an agent runs it. See [Sandboxes](#sandboxes). |
+| `pre setup claude [--yes] [--dry-run] [--local]` | For Claude Code. Run it in your own terminal. It writes the PRE skill to `.claude/skills/pre-onboard/`, and adds the API host and a remote Postgres host to Claude Code's sandbox network allowlist. It shows the change and asks first. It refuses to write when an agent runs it. See [Sandboxes](#sandboxes). |
 | `pre status` | Checks that the API answers and accepts the key, and names the workspace. Lists the datasets and models this folder made. |
 | `pre outcome define --name <slug> --sql <query> [--case account --cases accounts --event churn --event-past churned]` | Runs the outcome query and stores it. The query returns one row per case: the case id, then a 0/1 label. It refuses no rows, a label that is not 0 or 1, a repeated case id and a label that is always the same. The words go into every sentence the CLI prints. It guesses each word you leave out from the column names and says so. |
 | `pre outcome show` | Lists the stored outcomes and their queries. |
@@ -83,10 +83,13 @@ other column. It maps the hashes back to the case ids in memory.
 
 `pre setup` writes the smallest change it can. It merges into the file, so a second run changes nothing.
 
-It writes `.claude/settings.json`, key `sandbox.network.allowedDomains`. For a Postgres on this machine, `--local`
+It writes `.claude/settings.local.json`, key `sandbox.network.allowedDomains`. That is Claude Code's personal
+settings file; the team's `.claude/settings.json` stays as it is. Do not commit it. For a Postgres on this machine, `--local`
 also sets `sandbox.network.allowLocalBinding`, which opens every local port. It never sets `allowAllUnixSockets`.
-Claude Code is the only agent at launch; others are on the roadmap. The onboard skill comes with the plugin
-(`tools/agent-plugin`), not with `pre setup`.
+Claude Code is the only agent at launch; others are on the roadmap. It also writes the onboard skill to
+`.claude/skills/pre-onboard/SKILL.md`. It records the copy's SHA-256 in `.pre/config.json`, and later replaces only
+that copy: a skill you edited stays as it is. Commit the skill to share it with the team. Its source is
+`tools/agent-plugin`.
 
 These key names come from secondary sources. They live in `pre_cli/sandbox.py`.
 

{stratyfy_cli-0.2.0 → stratyfy_cli-0.2.1}/README.md

@@ -31,7 +31,7 @@ failure the command exits with 1 and prints `code`, `message` and `next_step`.
 | `pre guide` | Prints the method a coding agent follows, from set-up to the report. It is the plugin's `onboard` skill, bundled in the CLI. |
 | `pre init --api-key <key> [--api-url <url>]` | Writes a key you already have to `.pre/config.json`. The key can also come from `PRE_API_KEY`. |
 | `pre connect <db>` | Describes a SQLite or Postgres database: tables, row counts, columns, types, null share, sample values and foreign keys. It opens the database read-only. |
-| `pre setup claude [--yes] [--dry-run] [--local]` | For Claude Code. Run it in your own terminal. It adds the API host and a remote Postgres host to Claude Code's sandbox network allowlist. It shows the change and asks first. It refuses to write when an agent runs it. See [Sandboxes](#sandboxes). |
+| `pre setup claude [--yes] [--dry-run] [--local]` | For Claude Code. Run it in your own terminal. It writes the PRE skill to `.claude/skills/pre-onboard/`, and adds the API host and a remote Postgres host to Claude Code's sandbox network allowlist. It shows the change and asks first. It refuses to write when an agent runs it. See [Sandboxes](#sandboxes). |
 | `pre status` | Checks that the API answers and accepts the key, and names the workspace. Lists the datasets and models this folder made. |
 | `pre outcome define --name <slug> --sql <query> [--case account --cases accounts --event churn --event-past churned]` | Runs the outcome query and stores it. The query returns one row per case: the case id, then a 0/1 label. It refuses no rows, a label that is not 0 or 1, a repeated case id and a label that is always the same. The words go into every sentence the CLI prints. It guesses each word you leave out from the column names and says so. |
 | `pre outcome show` | Lists the stored outcomes and their queries. |
@@ -70,10 +70,13 @@ other column. It maps the hashes back to the case ids in memory.
 
 `pre setup` writes the smallest change it can. It merges into the file, so a second run changes nothing.
 
-It writes `.claude/settings.json`, key `sandbox.network.allowedDomains`. For a Postgres on this machine, `--local`
+It writes `.claude/settings.local.json`, key `sandbox.network.allowedDomains`. That is Claude Code's personal
+settings file; the team's `.claude/settings.json` stays as it is. Do not commit it. For a Postgres on this machine, `--local`
 also sets `sandbox.network.allowLocalBinding`, which opens every local port. It never sets `allowAllUnixSockets`.
-Claude Code is the only agent at launch; others are on the roadmap. The onboard skill comes with the plugin
-(`tools/agent-plugin`), not with `pre setup`.
+Claude Code is the only agent at launch; others are on the roadmap. It also writes the onboard skill to
+`.claude/skills/pre-onboard/SKILL.md`. It records the copy's SHA-256 in `.pre/config.json`, and later replaces only
+that copy: a skill you edited stays as it is. Commit the skill to share it with the team. Its source is
+`tools/agent-plugin`.
 
 These key names come from secondary sources. They live in `pre_cli/sandbox.py`.
 

{stratyfy_cli-0.2.0 → stratyfy_cli-0.2.1}/pre_cli/main.py

@@ -250,13 +250,14 @@ def try_sample(
         with gzip.open(file.payload) as src, part.open("wb") as dst:
             shutil.copyfileobj(src, dst)
         part.rename(db)
-    keep = {k: cfg[k] for k in ("api_url", "api_key", "key_id") if k in cfg}
-    config.save(config.load(folder / ".pre") | keep, folder / ".pre")
+    target = config.load(folder / ".pre") | {k: cfg[k] for k in ("api_url", "api_key", "key_id") if k in cfg}
+    if skill.status(folder, target.get("skill_sha256")) in ("missing", "older"):
+        target["skill_sha256"] = skill.install(folder)
+    config.save(target, folder / ".pre")
     words = {"name": name, "product": sample.product, "goal": sample.goal, "skill": skill.NAME}
     for file, template in (("README.md", SAMPLE_README), ("CLAUDE.md", SAMPLE_CLAUDE)):
         if not (path := folder / file).exists():
-            path.write_text(template.format(**words))
-    skill.install(folder)
+            path.write_text(template.format(**words), encoding="utf-8")
     step = f'Run `cd {folder}` and start Claude Code there. Then ask: "{sample.goal}"'
     result = {"folder": str(folder), "database": str(db), "product": sample.product, "goal": sample.goal}
     return result | {"next_step": step}, f"Wrote {db}. {sample.product}"
@@ -367,7 +368,7 @@ def setup(
     local: Annotated[bool, typer.Option("--local", help="Open local ports for a local Postgres.")] = False,
     json_out: JsonOut = False,
 ):
-    """Let a coding agent's sandbox reach the PRE API and the database."""
+    """Set up a coding agent in this folder: the PRE skill, and a sandbox that reaches the API and the database."""
     again = f"pre setup {agent}"
     if agent not in sandbox.AGENTS:
         raise CliError("bad_agent", f"`{again}` is not supported.", "Run `pre setup claude`.")
@@ -380,15 +381,24 @@ def setup(
     ask = not (yes or json_out or dry_run)
     if ask and plan["offer_local"] and typer.confirm("Also open every local port, so the sandbox reaches Postgres?"):
         plan = sandbox.plan(*args, True)
-    result, text = {k: v for k, v in plan.items() if k[0] != "_"} | {"written": False}, sandbox.describe(plan)
+    found = skill.status(Path.cwd(), cfg.get("skill_sha256"))
+    new_skill = str(skill.path(Path.cwd())) if found in ("missing", "older") else None
+    result = {k: v for k, v in plan.items() if k[0] != "_"} | {"skill": new_skill, "written": False}
+    text = sandbox.describe(plan) + (f"\nWrite the PRE skill to {new_skill}." if new_skill else "")
+    if found == "edited":
+        keep = f"{skill.path(Path.cwd())} differs from what pre wrote, so pre left it. Delete it to get the new one."
+        result["advice"], text = [*result["advice"], keep], f"{text}\n{keep}"
     restart = f"Restart {agent} in this folder, then ask it to run `pre status`."
-    if not plan["add"]:
+    if not plan["add"] and not new_skill:
         return result | {"next_step": restart}, text
     if dry_run or not (yes or ask):
         return result | {"next_step": f"Run `{again} --yes` in your own terminal to write it."}, text
     if ask:
         typer.confirm(f"{text}\nWrite this?", abort=True)
-    sandbox.apply(plan)
+    if plan["add"]:
+        sandbox.apply(plan)
+    if new_skill:
+        config.save(cfg | {"skill_sha256": skill.install(Path.cwd())})
     return result | {"written": True, "next_step": restart}, text if yes else "Written."
 
 

{stratyfy_cli-0.2.0 → stratyfy_cli-0.2.1}/pre_cli/sandbox.py

@@ -1,4 +1,4 @@
-"""pre setup: let Claude Code's sandbox reach the PRE API and the database. The plugin brings the skill."""
+"""pre setup: let Claude Code's sandbox reach the PRE API and the database. main.setup adds the skill."""
 
 import json
 import os
@@ -7,7 +7,8 @@ from urllib.parse import urlsplit
 
 # UNCONFIRMED. These names come from secondary sources, not Claude Code's own docs. Correct them here.
 # Other agents are on the roadmap (PLAN.md); each gets an entry here.
-AGENTS = {"claude": {"config": ".claude/settings.json", "env": ["CLAUDECODE"]}}
+# settings.local.json is Claude Code's personal file. A sandbox allowance belongs to one machine, not the team.
+AGENTS = {"claude": {"config": ".claude/settings.local.json", "env": ["CLAUDECODE"]}}
 CLAUDE_DOMAINS = "sandbox.network.allowedDomains"
 CLAUDE_LOCAL_BINDING = "sandbox.network.allowLocalBinding"
 CLAUDE_SOCKETS = "sandbox.network.allowAllUnixSockets"

stratyfy_cli-0.2.1/pre_cli/skill.py

@@ -0,0 +1,51 @@
+"""The onboarding skill: the method a coding agent follows with pre. Its one source is tools/agent-plugin. The
+wheel bundles it as pre_cli/skills/onboard/SKILL.md (hatch_build.py). In the repo, it is read from its source."""
+
+import hashlib
+from pathlib import Path
+
+HERE = Path(__file__).resolve().parent
+PATHS = (HERE / "skills" / "onboard" / "SKILL.md", HERE.parents[1] / "agent-plugin" / "skills" / "onboard" / "SKILL.md")
+NAME = "pre-onboard"  # in a project's .claude/skills, where a bare "onboard" could clash
+
+
+def text() -> str:
+    """SKILL.md with its front matter."""
+    return next(found for found in PATHS if found.exists()).read_text(encoding="utf-8")
+
+
+def body() -> str:
+    """SKILL.md without its front matter, to print."""
+    return text().split("\n---\n", 1)[1].lstrip()
+
+
+def path(project: Path) -> Path:
+    """Where Claude Code finds the skill in a project."""
+    return project / ".claude" / "skills" / NAME / "SKILL.md"
+
+
+def project_text() -> str:
+    return text().replace("\nname: onboard\n", f"\nname: {NAME}\n", 1)
+
+
+def digest(content: str) -> str:
+    return hashlib.sha256(content.encode("utf-8")).hexdigest()
+
+
+def status(project: Path, written: str | None) -> str:
+    """ "current", "missing", "older" (a copy pre wrote, unchanged since), or "edited" (anything else).
+
+    `written` is the digest of the copy pre last wrote here, from .pre/config.json.
+    """
+    if not path(project).exists():
+        return "missing"
+    found = digest(path(project).read_text(encoding="utf-8"))
+    return "current" if found == digest(project_text()) else "older" if found == written else "edited"
+
+
+def install(project: Path) -> str:
+    """Write the skill to the project's .claude/skills, where Claude Code finds it. Returns its digest."""
+    target = path(project)
+    target.parent.mkdir(parents=True, exist_ok=True)
+    target.write_text(project_text(), encoding="utf-8")
+    return digest(project_text())

{stratyfy_cli-0.2.0 → stratyfy_cli-0.2.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "stratyfy-cli"
-version = "0.2.0"
+version = "0.2.1"
 description = "The pre command: describe a database, train a PRE model, read its rules"
 readme = "README.md"
 requires-python = ">=3.11,<4"

stratyfy_cli-0.2.0/pre_cli/skill.py

@@ -1,26 +0,0 @@
-"""The onboarding skill: the method a coding agent follows with pre. Its one source is tools/agent-plugin. The
-wheel bundles it as pre_cli/skills/onboard/SKILL.md (hatch_build.py). In the repo, it is read from its source."""
-
-from pathlib import Path
-
-HERE = Path(__file__).resolve().parent
-PATHS = (HERE / "skills" / "onboard" / "SKILL.md", HERE.parents[1] / "agent-plugin" / "skills" / "onboard" / "SKILL.md")
-NAME = "pre-onboard"  # in a project's .claude/skills, where a bare "onboard" could clash
-
-
-def text() -> str:
-    """SKILL.md with its front matter."""
-    return next(path for path in PATHS if path.exists()).read_text()
-
-
-def body() -> str:
-    """SKILL.md without its front matter, to print."""
-    return text().split("\n---\n", 1)[1].lstrip()
-
-
-def install(project: Path) -> Path:
-    """Write the skill to the project's .claude/skills, where Claude Code finds it."""
-    path = project / ".claude" / "skills" / NAME / "SKILL.md"
-    path.parent.mkdir(parents=True, exist_ok=True)
-    path.write_text(text().replace("\nname: onboard\n", f"\nname: {NAME}\n", 1))
-    return path