skillctl-secure 0.30.3__tar.gz → 0.30.4__tar.gz

{skillctl_secure-0.30.3 → skillctl_secure-0.30.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: skillctl-secure
-Version: 0.30.3
+Version: 0.30.4
 Summary: Package manager, trust layer, and portability layer for Agent Skills
 Project-URL: Homepage, https://github.com/brian-scardina/skillctl
 Project-URL: Repository, https://github.com/brian-scardina/skillctl

{skillctl_secure-0.30.3 → skillctl_secure-0.30.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "skillctl-secure"
-version = "0.30.3"
+version = "0.30.4"
 description = "Package manager, trust layer, and portability layer for Agent Skills"
 readme = "README.md"
 license = "Apache-2.0"

skillctl_secure-0.30.4/skillctl/__init__.py

@@ -0,0 +1 @@
+__version__ = "0.30.4"

{skillctl_secure-0.30.3 → skillctl_secure-0.30.4}/skillctl/attack_surface_tui.py

@@ -38,7 +38,7 @@ class AttackSurfacePanel(Vertical):
                 with VerticalScroll(id='surface-details'):
                     yield Static(METHOD + '\n\nSelect a skill or indicator to see evidence and fixes. Shared copies appear under each destination but count once. Each run assesses at most 100 copies; remaining copies stay UNKNOWN.', id='surface-detail', markup=False)
             with Horizontal(id='surface-actions'):
-                yield Button('Assess installed skills', id='surface-assess', variant='primary', disabled=True)
+                yield Button('Reassess installed skills', id='surface-assess', variant='primary', disabled=True)
                 if self.auto_load:
                     yield Button('Close', id='surface-close')
 
@@ -49,13 +49,27 @@ class AttackSurfacePanel(Vertical):
     def ensure_loaded(self):
         if not self._loaded:
             self._loaded = True
-            self.load_surface()
+            self.start_assessment()
+
+    def start_assessment(self):
+        self._selection_made = False
+        self.query_one('#surface-assess', Button).disabled = True
+        self.query_one('#surface-detail', Static).update(METHOD + '\n\nAssessment in progress. Select a copy for current findings.')
+        self.load_surface(True, self.query_one('#surface-cves', Checkbox).value)
+
+    def on_checkbox_changed(self, event):
+        if event.checkbox.id == 'surface-cves':
+            event.stop()
+            if self._loaded:
+                self.start_assessment()
 
     @work(thread=True, exclusive=True, group='surface-assessment')
     def load_surface(self, assess=False, online=False):
         worker = get_current_worker()
         try:
             inventory = scan_inventory(self.manager)
+            if worker.is_cancelled:
+                return
             self.app.call_from_thread(self.show_surface, inventory, {}, not assess)
             if assess:
                 def progress(done, total, reports):
@@ -79,6 +93,8 @@ class AttackSurfacePanel(Vertical):
         surface = build_surface(inventory, reports)
         self.query_one('#surface-summary', Static).update(surface_summary(surface) + ('' if finished else '\nAssessing…'))
         self.query_one('#surface-assess', Button).disabled = not finished
+        if finished and not getattr(self, '_selection_made', False):
+            self.query_one('#surface-detail', Static).update(METHOD + '\n\nAssessment complete. Select a skill or indicator to see evidence and fixes.')
         tree = self.query_one('#surface-tree', Tree)
         tree.clear()
         tree.root.expand()
@@ -99,6 +115,7 @@ class AttackSurfacePanel(Vertical):
 
     def on_tree_node_selected(self, event):
         if event.node.data:
+            self._selection_made = True
             self.query_one('#surface-detail', Static).update(node_detail(event.node.data))
 
     def action_close(self):
@@ -110,9 +127,7 @@ class AttackSurfacePanel(Vertical):
         if event.button.id == 'surface-close':
             self.action_close()
         elif event.button.id == 'surface-assess':
-            event.button.disabled = True
-            self.query_one('#surface-detail', Static).update(METHOD + '\n\nAssessment in progress. Select a copy for current findings.')
-            self.load_surface(True, self.query_one('#surface-cves', Checkbox).value)
+            self.start_assessment()
 
 
 class AttackSurfaceScreen(ModalScreen):

{skillctl_secure-0.30.3 → skillctl_secure-0.30.4}/skillctl/inventory_audit_tui.py

@@ -22,11 +22,15 @@ class InventoryAuditScreen(FindingsScreen):
                 yield Button('Audit', id='findings-audit', variant='primary')
                 yield Button('Close', id='findings-close')
 
-    def on_mount(self):
-        pass
+    def on_mount(self, event):
+        # Keep this explicit local audit idle until the user presses Audit.
+        event.prevent_default()
 
     def on_button_pressed(self, event):
         event.stop()
+        # Textual also dispatches to base-class handlers unless prevented.
+        # This dialog audits a local copy, so FindingsScreen must not handle it.
+        event.prevent_default()
         if event.button.id == 'findings-close':
             self.dismiss()
         elif event.button.id == 'findings-audit':

{skillctl_secure-0.30.3 → skillctl_secure-0.30.4}/skillctl/tui_original.py

@@ -269,22 +269,22 @@ class SkillCtlTUI(App):
     def compose(self) -> ComposeResult:
         yield Header()
         with Horizontal(id="navigation"):
-            yield Tabs(*(Tab(label, id="section-" + key) for key, label in (("home", "Home"), ("skills", "Skills"), ("risk", "Attack Surface & Risk"), ("reviews", "Reviews"), ("organize", "Organize"), ("system", "System"))), id="sections")
+            yield Tabs(*(Tab(label, id="section-" + key) for key, label in (("home", "Home"), ("skills", "Skills"), ("risk", "Risks"), ("reviews", "Reviews"), ("organize", "Organize"), ("system", "System"))), id="sections")
             yield Input(placeholder="Search…  [/]", id="search", tooltip="Search skills, descriptions, categories, and collections. Press / to focus.")
         with Horizontal(id="body"):
             with TabbedContent(id="catalog"):
-                with TabPane("Home", id="home"):
+                with TabPane("Overview", id="home"):
                     yield DataTable(id="table-home", cursor_type="row")
                 with TabPane("Stats", id="stats"):
                     with VerticalScroll():
                         yield Static("", id="summary", markup=True)
-                with TabPane("Attack Surface & Risk", id="attack-surface"):
+                with TabPane("Risks", id="attack-surface"):
                     from .attack_surface_tui import AttackSurfacePanel
                     yield AttackSurfacePanel(self.manager, auto_load=False)
                 with TabPane("Available", id="available"):
                     yield DataTable(id="table-available", cursor_type="row")
                 with TabPane("Inventory", id="inventory"):
-                    yield Button("Attack surface & risk", id="btn-attack-surface")
+                    yield Button("Risks", id="btn-attack-surface")
                     yield Static("Discovering installed skills…", id="inventory-summary", markup=False)
                     yield DataTable(id="table-inventory", cursor_type="row")
                 with TabPane("Installed here", id="installed"):

{skillctl_secure-0.30.3 → skillctl_secure-0.30.4}/skillctl/tui_progressive.py

@@ -53,6 +53,8 @@ def install_progressive_startup(SkillCtlTUI, legacy_on_mount) -> None:
             pass
         # Start audit only after the primary catalog is visible and interactive.
         self.set_timer(0.05, self._auto_audit_catalog)
+        from .attack_surface_tui import AttackSurfacePanel
+        self.query_one(AttackSurfacePanel).ensure_loaded()
 
     def progressive_on_mount(self) -> None:
         self._primary_ready = False

skillctl_secure-0.30.3/skillctl/__init__.py

@@ -1 +0,0 @@
-__version__ = "0.30.3"