subedit 0.1.3__tar.gz → 0.1.4__tar.gz

{subedit-0.1.3 → subedit-0.1.4}/.claude-plugin/marketplace.json

@@ -1,10 +1,11 @@
 {
   "name": "mroops",
   "owner": { "name": "mroops0111" },
+  "description": "Plugins built alongside subedit.",
   "plugins": [
     {
       "name": "subedit",
-      "source": { "source": "github", "repo": "mroops0111/subedit" },
+      "source": "./plugins/subedit",
       "description": "Repairs comment layout silently on write, and returns only what needs rewriting."
     }
   ]

{subedit-0.1.3 → subedit-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: subedit
-Version: 0.1.3
+Version: 0.1.4
 Summary: A linter and formatter for the prose inside code comments, across languages.
 Project-URL: Homepage, https://github.com/mroops0111/subedit
 Project-URL: Repository, https://github.com/mroops0111/subedit

{subedit-0.1.3 → subedit-0.1.4/plugins/subedit}/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "subedit",
   "displayName": "subedit",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Repairs comment layout silently on write, and returns only what needs rewriting.",
   "author": {
     "name": "mroops0111"

{subedit-0.1.3 → subedit-0.1.4/plugins/subedit}/hooks/hooks.json

@@ -7,7 +7,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "${CLAUDE_PLUGIN_ROOT}/hooks/run-subedit-hook.sh",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}\"/scripts/run-subedit-hook.sh",
             "timeout": 30
           }
         ]
@@ -18,7 +18,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "${CLAUDE_PLUGIN_ROOT}/hooks/run-subedit-hook.sh prepare",
+            "command": "\"${CLAUDE_PLUGIN_ROOT}\"/scripts/run-subedit-hook.sh prepare",
             "timeout": 10
           }
         ]

{subedit-0.1.3/hooks → subedit-0.1.4/plugins/subedit/scripts}/run-subedit-hook.sh

@@ -4,7 +4,7 @@
 # because the first run installs subedit and downloads Vale, which no edit should wait on.
 set -uo pipefail
 
-SPEC='subedit[vale]>=0.1.3'
+SPEC='subedit[vale]>=0.1.4'
 # A bare uvx takes the first Python it finds, which on a stock Mac is too old for subedit.
 PYTHON='>=3.11'
 

{subedit-0.1.3 → subedit-0.1.4/plugins/subedit}/skills/judging-findings/SKILL.md

@@ -41,4 +41,4 @@ An error always needs a rewrite. The message states the fix, and these notes cov
 - `signature-restating-tag` and `subedit.SignatureTag`: delete the tag, since the signature already says it.
 - `subedit.EditHistory`: state the current behavior, not its history.
 - `subedit.Semicolon`, `subedit.EmDash`: split into two sentences, or use a comma or parentheses.
-- `subedit.Ellipsis`: give two or three concrete examples, and end the list there.
+- `subedit.Ellipsis`: name two or three concrete examples after "such as", as in "a transport failure, such as a connect, timeout, or DNS error". Dropping the trailing word and joining the rest with "or" turns a sample into the whole set, which narrows what the sentence claims.

{subedit-0.1.3 → subedit-0.1.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "subedit"
-version = "0.1.3"
+version = "0.1.4"
 description = "A linter and formatter for the prose inside code comments, across languages."
 authors = [
     {name = "Yun-Tai Yang"},

{subedit-0.1.3 → subedit-0.1.4}/src/subedit/__init__.py

@@ -1,3 +1,3 @@
-__version__ = '0.1.3'
+__version__ = '0.1.4'
 
 __all__ = ['__version__']

subedit-0.1.4/src/subedit/vale/styles/subedit/Ellipsis.yml

@@ -0,0 +1,9 @@
+extends: existence
+message: "Trailing '%s'. Name two or three concrete examples after 'such as', which keeps the list open without trailing off."
+level: error
+ignorecase: true
+tokens:
+  - '\.\.\.'
+  - '…'
+  - '\band so on\b'
+  - '\betc\b'

{subedit-0.1.3 → subedit-0.1.4}/tests/test_plugin.py

@@ -9,10 +9,12 @@ from subedit.presentation.harnesses.claude_code_harness import EDIT_TOOLS
 
 
 ROOT = pathlib.Path(__file__).resolve().parents[1]
-MANIFEST = ROOT / '.claude-plugin' / 'plugin.json'
+PLUGIN = ROOT / 'plugins' / 'subedit'
+MANIFEST = PLUGIN / '.claude-plugin' / 'plugin.json'
 MARKETPLACE = ROOT / '.claude-plugin' / 'marketplace.json'
-HOOKS = ROOT / 'hooks' / 'hooks.json'
-WRAPPER = ROOT / 'hooks' / 'run-subedit-hook.sh'
+HOOKS = PLUGIN / 'hooks' / 'hooks.json'
+WRAPPER = PLUGIN / 'scripts' / 'run-subedit-hook.sh'
+WRAPPER_COMMAND = f'"${{CLAUDE_PLUGIN_ROOT}}"/scripts/{WRAPPER.name}'
 
 
 def load(path: pathlib.Path) -> typing.Any:
@@ -45,8 +47,7 @@ def test_the_hook_matcher_covers_every_tool_the_harness_recognises() -> None:
 def test_the_hook_runs_the_wrapper_from_the_plugin_directory() -> None:
     command = load(HOOKS)['hooks']['PostToolUse'][0]['hooks'][0]['command']
 
-    assert '${CLAUDE_PLUGIN_ROOT}' in command, 'a bare path breaks once the plugin is installed elsewhere'
-    assert command.endswith(WRAPPER.name)
+    assert command == WRAPPER_COMMAND, 'a bare or unquoted path breaks once the plugin is installed elsewhere'
 
 
 def test_the_wrapper_is_executable() -> None:
@@ -83,18 +84,24 @@ def test_the_wrapper_runs_the_released_version() -> None:
 def test_a_session_start_warms_the_same_wrapper() -> None:
     command = load(HOOKS)['hooks']['SessionStart'][0]['hooks'][0]['command']
 
-    assert command == f'${{CLAUDE_PLUGIN_ROOT}}/hooks/{WRAPPER.name} prepare'
+    assert command == f'{WRAPPER_COMMAND} prepare'
 
 
-def test_the_marketplace_points_at_this_repository() -> None:
+def test_the_marketplace_points_at_the_plugin_directory() -> None:
     listed = load(MARKETPLACE)['plugins']
 
     assert [plugin['name'] for plugin in listed] == ['subedit']
-    assert listed[0]['source']['repo'] == 'mroops0111/subedit'
+    assert (ROOT / listed[0]['source']).resolve() == PLUGIN, 'a relative source resolves from the marketplace root'
+
+
+def test_the_plugin_carries_everything_it_runs() -> None:
+    """An install copies only the plugin directory, so nothing it loads may live outside it."""
+    assert {path.name for path in PLUGIN.iterdir()} == {'.claude-plugin', 'hooks', 'scripts', 'skills'}
+    assert [path.name for path in (PLUGIN / '.claude-plugin').iterdir()] == ['plugin.json']
 
 
 def test_the_judging_skill_names_itself_as_its_folder_does() -> None:
-    skill = ROOT / 'skills' / 'judging-findings' / 'SKILL.md'
+    skill = PLUGIN / 'skills' / 'judging-findings' / 'SKILL.md'
     front_matter = skill.read_text(encoding='utf-8').split('---')[1]
 
     assert 'name: judging-findings' in front_matter, 'Claude Code loads a skill by the name in its front matter'

{subedit-0.1.3 → subedit-0.1.4}/uv.lock

@@ -349,7 +349,7 @@ wheels = [
 
 [[package]]
 name = "subedit"
-version = "0.1.3"
+version = "0.1.4"
 source = { editable = "." }
 dependencies = [
     { name = "autocorrect-py" },

subedit-0.1.3/src/subedit/vale/styles/subedit/Ellipsis.yml

@@ -1,9 +0,0 @@
-extends: existence
-message: "Trailing '%s'. Give two or three concrete examples and stop."
-level: error
-ignorecase: true
-tokens:
-  - '\.\.\.'
-  - '…'
-  - '\band so on\b'
-  - '\betc\b'