compartment 2__tar.gz → 2.1__tar.gz

{compartment-2 → compartment-2.1}/PKG-INFO

@@ -1,7 +1,30 @@
 Metadata-Version: 2.4
 Name: compartment
-Version: 2
+Version: 2.1
 Summary: Compartment - high-security, fully offline, encrypted vector memory for AI agents (MCP)
+Project-URL: Homepage, https://github.com/MaxFreedomPollard/Compartment
+Project-URL: Source, https://github.com/MaxFreedomPollard/Compartment
+Project-URL: Issues, https://github.com/MaxFreedomPollard/Compartment/issues
+Project-URL: Changelog, https://github.com/MaxFreedomPollard/Compartment/releases
+Project-URL: Documentation, https://github.com/MaxFreedomPollard/Compartment#readme
+Project-URL: Security, https://github.com/MaxFreedomPollard/Compartment/blob/main/SECURITY.md
+Keywords: mcp,model-context-protocol,memory,agent-memory,vector-database,semantic-search,encryption,offline,local-first,claude,claude-code,privacy,rag
+Classifier: Development Status :: 4 - Beta
+Classifier: Intended Audience :: Developers
+Classifier: Environment :: Console
+Classifier: Environment :: MacOS X
+Classifier: Operating System :: OS Independent
+Classifier: Operating System :: MacOS :: MacOS X
+Classifier: Operating System :: Microsoft :: Windows
+Classifier: Operating System :: POSIX :: Linux
+Classifier: Programming Language :: Python :: 3
+Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
+Classifier: Topic :: Security :: Cryptography
+Classifier: Topic :: Scientific/Engineering :: Artificial Intelligence
+Classifier: Topic :: Database :: Database Engines/Servers
+Classifier: Typing :: Typed
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
 Requires-Dist: pynacl>=1.5

{compartment-2 → compartment-2.1}/pyproject.toml

@@ -4,10 +4,31 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "compartment"
-version = "2"
+version = "2.1"
 description = "Compartment - high-security, fully offline, encrypted vector memory for AI agents (MCP)"
 readme = "README.md"
 requires-python = ">=3.11"
+keywords = ["mcp", "model-context-protocol", "memory", "agent-memory",
+            "vector-database", "semantic-search", "encryption", "offline",
+            "local-first", "claude", "claude-code", "privacy", "rag"]
+classifiers = [
+    "Development Status :: 4 - Beta",
+    "Intended Audience :: Developers",
+    "Environment :: Console",
+    "Environment :: MacOS X",
+    "Operating System :: OS Independent",
+    "Operating System :: MacOS :: MacOS X",
+    "Operating System :: Microsoft :: Windows",
+    "Operating System :: POSIX :: Linux",
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.11",
+    "Programming Language :: Python :: 3.12",
+    "Programming Language :: Python :: 3.13",
+    "Topic :: Security :: Cryptography",
+    "Topic :: Scientific/Engineering :: Artificial Intelligence",
+    "Topic :: Database :: Database Engines/Servers",
+    "Typing :: Typed",
+]
 dependencies = [
     "pynacl>=1.5",
     "argon2-cffi>=23.1",
@@ -17,6 +38,18 @@ dependencies = [
     "mcp>=1.0",
 ]
 
+# PyPI is the repo's declared homepage, so it is a real front door - and
+# without these it is a dead end: no Source link, no Issues link, nothing
+# carrying a reader back to GitHub. Directories that scrape PyPI were
+# producing a listing with no repository link at all.
+[project.urls]
+Homepage = "https://github.com/MaxFreedomPollard/Compartment"
+Source = "https://github.com/MaxFreedomPollard/Compartment"
+Issues = "https://github.com/MaxFreedomPollard/Compartment/issues"
+Changelog = "https://github.com/MaxFreedomPollard/Compartment/releases"
+Documentation = "https://github.com/MaxFreedomPollard/Compartment#readme"
+Security = "https://github.com/MaxFreedomPollard/Compartment/blob/main/SECURITY.md"
+
 [project.scripts]
 compartment = "compartment.cli:main"
 cmpt = "compartment.cli:main"

{compartment-2 → compartment-2.1}/src/compartment/__init__.py

@@ -1,6 +1,6 @@
 """Compartment - high-security, fully offline, encrypted vector memory for AI agents."""
 
-__version__ = "2"
+__version__ = "2.1"
 
 from . import offline_guard as _og
 

{compartment-2 → compartment-2.1}/src/compartment/menubar.py

@@ -396,6 +396,7 @@ def run(vault: str | None = None, show: bool = False,
                             NSRunningApplication,
                             NSFont, NSImage, NSMakeRect, NSMenu, NSMenuItem,
                             NSPanel, NSPopover, NSScreen, NSSegmentedControl,
+                            NSClipView, NSScrollView,
                             NSSecureTextField, NSStackView, NSStatusBar,
                             NSSwitch, NSTextField,
                             NSButton, NSView, NSViewController,
@@ -468,6 +469,17 @@ def run(vault: str | None = None, show: bool = False,
             CONTENT_WIDTH).setActive_(True)
         return b
 
+    class TopClip(NSClipView):
+        """A clip view whose origin is its top left corner.
+
+        AppKit's default is the bottom left, which inside a scroll view means
+        the panel opens showing its last line, with the title somewhere above
+        the fold. Flipping it makes "scrolled to the top" the resting state.
+        """
+
+        def isFlipped(self):
+            return True
+
     class Controller(NSObject):
         def init(self):
             self = objc.super(Controller, self).init()
@@ -485,8 +497,78 @@ def run(vault: str | None = None, show: bool = False,
             return self
 
         # ---- building the popover contents -----------------------------
+        @objc.python_method
+        def _finish(self, views):
+            stack = NSStackView.stackViewWithViews_(views)
+            stack.setOrientation_(_VERTICAL)
+            stack.setAlignment_(_LEADING)
+            stack.setSpacing_(6)
+            stack.setEdgeInsets_((CONTENT_INSET, CONTENT_INSET,
+                                  CONTENT_INSET, CONTENT_INSET))
+            stack.layoutSubtreeIfNeeded()
+            fit = stack.fittingSize()
+            stack.setFrameSize_((POPOVER_WIDTH, fit.height))
+            if fit.height <= POPOVER_MAX_HEIGHT:
+                return stack
+
+            # Taller than a popover is allowed to be. Scroll it. What this
+            # did before was cut the overflow off and say nothing, so the
+            # buttons below the fold were not merely out of reach - there
+            # was no sign they existed. That is how the passphrase form
+            # came to show one box out of two.
+            stack.setTranslatesAutoresizingMaskIntoConstraints_(True)
+            box = NSMakeRect(0, 0, POPOVER_WIDTH, POPOVER_MAX_HEIGHT)
+            scroll = NSScrollView.alloc().initWithFrame_(box)
+            scroll.setContentView_(TopClip.alloc().initWithFrame_(box))
+            scroll.setHasVerticalScroller_(True)
+            scroll.setDrawsBackground_(False)
+            scroll.setDocumentView_(stack)
+            return scroll
+
+        @objc.python_method
+        def buildChangeBody(self, st):
+            """Changing the passphrase gets the whole panel to itself.
+
+            The ordinary panel already fills POPOVER_MAX_HEIGHT, and this
+            popover clips instead of scrolling. Appending the fields to the
+            bottom of it put the second box, Save, and the error note below
+            the visible edge: you saw one box, pressed Return, and the save
+            failed its repeat check against an empty field you could neither
+            fill nor read the complaint about. A focused view cannot overflow.
+            """
+            def secure(placeholder):
+                f = NSSecureTextField.alloc().initWithFrame_(
+                    NSMakeRect(0, 0, CONTENT_WIDTH, 24))
+                f.setPlaceholderString_(placeholder)
+                f.setTarget_(self)
+                f.setAction_("saveChangePw:")     # Return saves
+                return f
+
+            self.pw_new = secure("New passphrase")
+            self.pw_repeat = secure("Repeat it")
+            views = [row(label("Change password", 15, bold=True), _spacer()),
+                     label(summarise(st), 11, secondary=True),
+                     divider(),
+                     row(self.pw_new),
+                     row(self.pw_repeat)]
+            if self.change_note:
+                views.append(label(self.change_note, 11, secondary=True,
+                                   wrap=True))
+            views.append(label("Both boxes must match. There is no recovery "
+                               "phrase - if you forget this, the memories are "
+                               "unrecoverable.", 10, secondary=True, wrap=True))
+            save_b = NSButton.buttonWithTitle_target_action_("Save", self,
+                                                             "saveChangePw:")
+            save_b.setKeyEquivalent_("\r")
+            cancel_b = NSButton.buttonWithTitle_target_action_(
+                "Cancel", self, "cancelChangePw:")
+            views.append(row(save_b, cancel_b, _spacer()))
+            return self._finish(views)
+
         def buildBody(self):
             st = self.state
+            if self.changing_pw and st["exists"] and not st["locked"]:
+                return self.buildChangeBody(st)
             views = []
             title = row(label("Compartment", 15, bold=True),
                         label("locked" if st["locked"] else "unlocked", 11,
@@ -516,33 +598,11 @@ def run(vault: str | None = None, show: bool = False,
                 views.append(label("Stays unlocked until restart or Lock",
                                    10, secondary=True))
 
-            # Changing the passphrase needs the vault open, because rekey
-            # re-wraps a master key it can only hold while unlocked.
+            # Changing the passphrase is a separate view: see buildChangeBody.
             self.pw_new = self.pw_repeat = None
-            if st["exists"] and not st["locked"] and self.changing_pw:
-                def secure(placeholder):
-                    f = NSSecureTextField.alloc().initWithFrame_(
-                        NSMakeRect(0, 0, CONTENT_WIDTH - 92, 24))
-                    f.setPlaceholderString_(placeholder)
-                    f.setTarget_(self)
-                    f.setAction_("saveChangePw:")     # Return saves
-                    return f
-
-                self.pw_new = secure("New passphrase")
-                self.pw_repeat = secure("Repeat it")
-                save_b = NSButton.buttonWithTitle_target_action_(
-                    "Save", self, "saveChangePw:")
-                cancel_b = NSButton.buttonWithTitle_target_action_(
-                    "Cancel", self, "cancelChangePw:")
-                views.append(self.pw_new)
-                views.append(row(self.pw_repeat, save_b))
-                views.append(row(cancel_b, _spacer()))
-                if self.change_note:
-                    views.append(label(self.change_note, 11, secondary=True,
-                                       wrap=True))
-                views.append(label("There is no recovery phrase. If you forget "
-                                   "this, the memories are unrecoverable.",
-                                   10, secondary=True, wrap=True))
+            if self.change_note:
+                views.append(label(self.change_note, 11, secondary=True,
+                                   wrap=True))
             views.append(divider())
 
             views.append(label("SETTINGS", 10, bold=True, secondary=True))
@@ -613,17 +673,7 @@ def run(vault: str | None = None, show: bool = False,
             else:
                 views.append(row(refresh, _spacer(), quit_b))
 
-            stack = NSStackView.stackViewWithViews_(views)
-            stack.setOrientation_(_VERTICAL)
-            stack.setAlignment_(_LEADING)
-            stack.setSpacing_(6)
-            stack.setEdgeInsets_((CONTENT_INSET, CONTENT_INSET,
-                                  CONTENT_INSET, CONTENT_INSET))
-            stack.layoutSubtreeIfNeeded()
-            fit = stack.fittingSize()
-            stack.setFrameSize_((POPOVER_WIDTH,
-                                 min(fit.height, POPOVER_MAX_HEIGHT)))
-            return stack
+            return self._finish(views)
 
         # Not an action: PyObjC infers a selector from the method name, and
         # a name with no trailing underscores means "takes no arguments".

{compartment-2 → compartment-2.1}/src/compartment/systray.py

@@ -152,6 +152,18 @@ def set_login(enabled: bool) -> str:
         return f"error: {exc}"
 
 
+def panel_geometry(content: int, maximum: int) -> tuple[int, bool]:
+    """How tall to draw the panel, and whether it needs a scrollbar.
+
+    Split out from the Tk code so the rule is checked on every OS in CI: any
+    content taller than the panel scrolls. It is never cut off - the panel
+    losing its bottom silently is the whole reason this function exists.
+    """
+    if content > maximum:
+        return maximum, True
+    return max(content, 1), False
+
+
 # --- the app ---------------------------------------------------------------
 
 def run(vault: str | None = None, show: bool = False,
@@ -192,14 +204,53 @@ def run(vault: str | None = None, show: bool = False,
     root.withdraw()                                   # no stray empty window
     panel: dict = {"win": None, "note": None}
 
-    def build(win) -> None:
-        for child in win.winfo_children():
-            child.destroy()
-        state = fetch_state(vault_path)
-        frame = ttk.Frame(win, padding=14)
-        frame.pack(fill="both", expand=True)
+    def _content(frame, state) -> None:
+        """Everything the panel shows. Packs into `frame`, never sizes it -
+        deciding how tall the window gets is build()'s job, below."""
         s = state["settings"]
 
+        # Changing the passphrase gets the whole panel to itself, rather than
+        # being appended below a panel that is already full. Two boxes, Save
+        # and the reason the last attempt failed all belong on screen at once.
+        if state["exists"] and not state["locked"] and panel.get("changing"):
+            ttk.Label(frame, text="Change password", wraplength=WRAP,
+                      font=("Segoe UI", 10, "bold")).pack(anchor="w")
+            ttk.Label(frame, text=summarise(state), wraplength=WRAP,
+                      foreground="#666").pack(anchor="w", pady=(2, 0))
+            ttk.Separator(frame).pack(fill="x", pady=10)
+            new = ttk.Entry(frame, show="•", width=34)
+            new.pack(anchor="w")
+            rep = ttk.Entry(frame, show="•", width=34)
+            rep.pack(anchor="w", pady=(6, 0))
+            new.focus_set()
+
+            def do_change(*_):
+                ok, note = change_passphrase(vault_path, new.get(), rep.get())
+                new.delete(0, "end")          # never leave one on screen
+                rep.delete(0, "end")
+                panel["changing"] = not ok
+                panel["change_note"] = note
+                refresh()
+
+            new.bind("<Return>", do_change)
+            rep.bind("<Return>", do_change)
+            if panel.get("change_note"):
+                ttk.Label(frame, text=panel["change_note"], wraplength=WRAP,
+                          foreground="#b00020").pack(anchor="w", pady=(8, 0))
+            ttk.Label(frame, text="Both boxes must match. There is no recovery "
+                                  "phrase - if you forget this, the memories "
+                                  "are unrecoverable.",
+                      foreground="#666", wraplength=WRAP,
+                      justify="left").pack(anchor="w", pady=(8, 0))
+            bar = ttk.Frame(frame)
+            bar.pack(fill="x", pady=(10, 0))
+            ttk.Button(bar, text="Save", command=do_change).pack(side="left")
+            ttk.Button(bar, text="Cancel",
+                       command=lambda: (panel.update(changing=False,
+                                                     change_note=None),
+                                        refresh())).pack(side="left", padx=6)
+            return
+
         ttk.Label(frame, text=summarise(state), wraplength=WRAP,
                   font=("Segoe UI", 10, "bold")).pack(anchor="w")
         if state.get("error"):
@@ -275,43 +326,7 @@ def run(vault: str | None = None, show: bool = False,
                       wraplength=WRAP,
                       justify="left").pack(anchor="w", pady=(4, 0))
 
-        if state["exists"] and not state["locked"] and panel.get("changing"):
-            ttk.Separator(frame).pack(fill="x", pady=10)
-            ttk.Label(frame, text="CHANGE PASSWORD",
-                      font=("Segoe UI", 8, "bold")).pack(anchor="w")
-            new = ttk.Entry(frame, show="•", width=34)
-            new.pack(anchor="w", pady=(6, 0))
-            rep = ttk.Entry(frame, show="•", width=34)
-            rep.pack(anchor="w", pady=(4, 0))
-            new.focus_set()
-
-            def do_change(*_):
-                ok, note = change_passphrase(vault_path, new.get(), rep.get())
-                # Never leave a passphrase sitting in a field on screen.
-                new.delete(0, "end")
-                rep.delete(0, "end")
-                panel["changing"] = not ok
-                panel["change_note"] = note
-                refresh()
-
-            new.bind("<Return>", do_change)
-            rep.bind("<Return>", do_change)
-            bar = ttk.Frame(frame)
-            bar.pack(fill="x", pady=(6, 0))
-            ttk.Button(bar, text="Save", command=do_change).pack(side="left")
-            ttk.Button(bar, text="Cancel",
-                       command=lambda: (panel.update(changing=False,
-                                                     change_note=None),
-                                        refresh())).pack(side="left", padx=6)
-            if panel.get("change_note"):
-                ttk.Label(frame, text=panel["change_note"],
-                          wraplength=WRAP).pack(anchor="w",
-                                                            pady=(4, 0))
-            ttk.Label(frame, text="There is no recovery phrase. If you forget "
-                                  "this, the memories are unrecoverable.",
-                      foreground="#666", wraplength=WRAP,
-                      justify="left").pack(anchor="w", pady=(4, 0))
-        elif panel.get("change_note"):
+        if panel.get("change_note"):        # result of the last attempt
             ttk.Label(frame, text=panel["change_note"],
                       wraplength=WRAP).pack(anchor="w", pady=(8, 0))
 
@@ -332,10 +347,56 @@ def run(vault: str | None = None, show: bool = False,
                                             refresh())).pack(side="left", padx=6)
         ttk.Button(buttons, text="Quit", command=quit_app).pack(side="right")
 
+    def build(win) -> None:
+        """Fill the window, and give it a scrollbar if the content overflows.
+
+        The content goes inside a canvas rather than straight into the window.
+        A fixed-height window cut its own bottom off and said nothing about
+        it, which is how the passphrase form came to show one box out of two:
+        the second box and the Save button were not merely out of reach, there
+        was no sign on screen that they existed at all.
+        """
+        for child in win.winfo_children():
+            child.destroy()
+        state = fetch_state(vault_path)
+
+        canvas = tk.Canvas(win, highlightthickness=0, borderwidth=0, width=PW)
+        try:                                  # match the themed background
+            canvas.configure(background=ttk.Style().lookup("TFrame",
+                                                           "background"))
+        except tk.TclError:                   # a theme without one: leave it
+            pass
+        vbar = ttk.Scrollbar(win, orient="vertical", command=canvas.yview)
+        canvas.configure(yscrollcommand=vbar.set)
+        canvas.pack(side="left", fill="both", expand=True)
+        frame = ttk.Frame(canvas, padding=14)
+        canvas.create_window((0, 0), window=frame, anchor="nw", width=PW)
+
+        _content(frame, state)
+
+        frame.update_idletasks()
+        need = frame.winfo_reqheight()
+        height, scrolling = panel_geometry(need, PMH)
+        canvas.configure(height=height, scrollregion=(0, 0, PW, need))
+        if scrolling:
+            vbar.pack(side="right", fill="y")
+            # Bound on the window, not the canvas: the content covers the
+            # canvas, so the wheel event never reaches it. Windows reports
+            # the delta in multiples of 120.
+            win.bind("<MouseWheel>",
+                     lambda e: canvas.yview_scroll(-(e.delta // 120), "units"))
+            vbar.update_idletasks()
+            panel["scroll_w"] = vbar.winfo_reqwidth()
+        else:
+            win.unbind("<MouseWheel>")
+            panel["scroll_w"] = 0
+
     def place(win) -> None:
         """Bottom right, above the taskbar, where the tray icon is."""
         win.update_idletasks()
-        w = PW
+        # The scrollbar sits beside the content, so widen the window by it
+        # rather than letting it eat a strip off the right of every line.
+        w = PW + panel.get("scroll_w", 0)
         h = min(win.winfo_reqheight(), PMH)
         x = win.winfo_screenwidth() - w - 12
         y = win.winfo_screenheight() - h - TBM

{compartment-2 → compartment-2.1}/src/compartment.egg-info/PKG-INFO

@@ -1,7 +1,30 @@
 Metadata-Version: 2.4
 Name: compartment
-Version: 2
+Version: 2.1
 Summary: Compartment - high-security, fully offline, encrypted vector memory for AI agents (MCP)
+Project-URL: Homepage, https://github.com/MaxFreedomPollard/Compartment
+Project-URL: Source, https://github.com/MaxFreedomPollard/Compartment
+Project-URL: Issues, https://github.com/MaxFreedomPollard/Compartment/issues
+Project-URL: Changelog, https://github.com/MaxFreedomPollard/Compartment/releases
+Project-URL: Documentation, https://github.com/MaxFreedomPollard/Compartment#readme
+Project-URL: Security, https://github.com/MaxFreedomPollard/Compartment/blob/main/SECURITY.md
+Keywords: mcp,model-context-protocol,memory,agent-memory,vector-database,semantic-search,encryption,offline,local-first,claude,claude-code,privacy,rag
+Classifier: Development Status :: 4 - Beta
+Classifier: Intended Audience :: Developers
+Classifier: Environment :: Console
+Classifier: Environment :: MacOS X
+Classifier: Operating System :: OS Independent
+Classifier: Operating System :: MacOS :: MacOS X
+Classifier: Operating System :: Microsoft :: Windows
+Classifier: Operating System :: POSIX :: Linux
+Classifier: Programming Language :: Python :: 3
+Classifier: Programming Language :: Python :: 3.11
+Classifier: Programming Language :: Python :: 3.12
+Classifier: Programming Language :: Python :: 3.13
+Classifier: Topic :: Security :: Cryptography
+Classifier: Topic :: Scientific/Engineering :: Artificial Intelligence
+Classifier: Topic :: Database :: Database Engines/Servers
+Classifier: Typing :: Typed
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
 Requires-Dist: pynacl>=1.5

{compartment-2 → compartment-2.1}/tests/test_systray.py

@@ -152,3 +152,33 @@ def test_login_helpers_report_instead_of_raising_off_windows():
     # No winreg here, so both must degrade to a message, not an exception.
     assert isinstance(systray.login_status(), str)
     assert systray.set_login(True).startswith("error:")
+
+
+# The panel used to draw itself at min(content, PANEL_MAX_HEIGHT) and simply
+# lose whatever fell below the line - no scrollbar, no ellipsis, nothing. The
+# change-password form was the casualty: it rendered one of its two boxes,
+# and the Save button and the "the two entries do not match" complaint were
+# both below the fold, so the form looked like it worked and changed nothing.
+def test_content_that_fits_is_drawn_whole_and_needs_no_scrollbar():
+    h, scrolling = systray.panel_geometry(300, systray.PANEL_MAX_HEIGHT)
+    assert (h, scrolling) == (300, False)
+
+
+def test_content_taller_than_the_panel_scrolls_rather_than_being_cut_off():
+    tall = systray.PANEL_MAX_HEIGHT * 3
+    h, scrolling = systray.panel_geometry(tall, systray.PANEL_MAX_HEIGHT)
+    assert scrolling, "overflow must scroll, never be silently clipped"
+    assert h == systray.PANEL_MAX_HEIGHT
+
+
+def test_the_panel_never_exceeds_its_maximum():
+    for content in (1, 100, 639, 640, 641, 5000):
+        h, _ = systray.panel_geometry(content, systray.PANEL_MAX_HEIGHT)
+        assert 0 < h <= systray.PANEL_MAX_HEIGHT, content
+
+
+def test_exactly_full_height_does_not_scroll():
+    # The boundary: equal to the maximum still fits, so no scrollbar appears.
+    assert systray.panel_geometry(systray.PANEL_MAX_HEIGHT,
+                                  systray.PANEL_MAX_HEIGHT) == (
+        systray.PANEL_MAX_HEIGHT, False)