ompchamber 3.4.0 → 3.6.0

package/src/client/components/settings/categories/skill-settings/DetailPane.tsx

@@ -1,5 +1,5 @@
 import { useEffect, useState } from 'preact/hooks';
-import { Check, Eye, EyeOff, Info, Save, Sparkles, Trash2, User } from 'lucide-preact';
+import { Check, Eye, EyeOff, Info, RefreshCw, Save, Sparkles, Trash2, User } from 'lucide-preact';
 import type { SkillItem } from '@/shared/types';
 
 interface SkillDetailPaneProps {
@@ -8,6 +8,18 @@ interface SkillDetailPaneProps {
   onSave: (skillData: Partial<SkillItem>) => void;
   onDelete?: (id: string) => void;
   onOpenCatalog?: () => void;
+  onReload?: () => void;
+  isReloading?: boolean;
+  /** Disabled when no workspace is selected: there is no `.omp/skills` to
+   *  write a project skill into. */
+  projectScopeDisabled?: boolean;
+  /**
+   * Root a NEW skill lands in, from the scope the panel is showing. Without
+   * this a create made while a workspace is selected wrote to the user root —
+   * the skill then appeared in a list the panel was not showing, and the
+   * workspace's own `.omp/skills` never received it.
+   */
+  defaultLocation?: 'user' | 'project';
 }
 
 export function SkillDetailPane({
@@ -16,33 +28,45 @@ export function SkillDetailPane({
   onSave,
   onDelete,
   onOpenCatalog,
+  onReload,
+  isReloading = false,
+  projectScopeDisabled = false,
+  defaultLocation = 'user',
 }: SkillDetailPaneProps) {
   const [name, setName] = useState('');
   const [description, setDescription] = useState('');
   const [location, setLocation] = useState<'user' | 'project'>('user');
-  const [locationLabel, setLocationLabel] = useState('User / OpenCode');
+  const [locationLabel, setLocationLabel] = useState('User / omp agent');
   const [instructions, setInstructions] = useState('');
+  const [hidden, setHidden] = useState(false);
   const [savedToast, setSavedToast] = useState(false);
   const [showLineNumbers, setShowLineNumbers] = useState(true);
 
+  // A skill the chamber does not own (a Claude plugin, a registry package) is
+  // shown read-only: its file belongs to another tool, and a save here would
+  // rewrite it. `managed` is absent on a brand-new draft, which is editable.
+  const readOnly = !isCreatingNew && skill != null && skill.managed !== true;
+
   useEffect(() => {
     if (isCreatingNew || !skill) {
       setName('new-skill');
       setDescription('');
-      setLocation('user');
-      setLocationLabel('User / OpenCode');
+      setLocation(defaultLocation);
+      setLocationLabel(defaultLocation === 'project' ? 'Project / .omp/skills' : 'User / omp agent');
       setInstructions(`---
 description: ""
 ---
 `);
+      setHidden(false);
     } else {
       setName(skill.name);
       setDescription(skill.description);
       setLocation(skill.location);
-      setLocationLabel(skill.locationLabel || 'User / OpenCode');
+      setLocationLabel(skill.locationLabel || 'User / omp agent');
       setInstructions(skill.instructions);
+      setHidden(skill.hidden === true);
     }
-  }, [skill, isCreatingNew]);
+  }, [skill, isCreatingNew, defaultLocation]);
 
   const handleSave = () => {
     onSave({
@@ -51,6 +75,7 @@ description: ""
       location,
       locationLabel,
       instructions,
+      hidden,
     });
     setSavedToast(true);
     setTimeout(() => setSavedToast(false), 2000);
@@ -68,22 +93,44 @@ description: ""
             {isCreatingNew ? 'New Skill' : name}
           </h2>
           <p className="text-xs text-ink/60 mt-0.5">
-            {isCreatingNew ? 'Configure a new skill' : 'Configure skill details and prompt rules'}
+            {isCreatingNew ? 'Configure a new skill' : `Configure skill details and prompt rules · ${locationLabel}`}
           </p>
         </div>
 
-        {onOpenCatalog && (
-          <button
-            type="button"
-            onClick={onOpenCatalog}
-            className="flex items-center gap-1.5 px-3 py-1.5 rounded-lg border border-ink/20 hover:border-ink/40 text-xs font-medium text-ink hover:bg-ink/5 transition-colors cursor-pointer"
-          >
-            <Sparkles size={14} className="text-ink/70" />
-            <span>Browse Catalog</span>
-          </button>
-        )}
+        <div className="flex items-center gap-2">
+          {onReload && (
+            <button
+              type="button"
+              onClick={onReload}
+              disabled={isReloading}
+              title="Re-discover skills in the running agents"
+              className="flex items-center gap-1.5 px-3 py-1.5 rounded-lg border border-ink/20 hover:border-ink/40 text-xs font-medium text-ink hover:bg-ink/5 transition-colors cursor-pointer disabled:opacity-50"
+            >
+              <RefreshCw size={14} className={isReloading ? 'animate-spin text-ink/70' : 'text-ink/70'} />
+              <span>Reload</span>
+            </button>
+          )}
+          {onOpenCatalog && (
+            <button
+              type="button"
+              onClick={onOpenCatalog}
+              className="flex items-center gap-1.5 px-3 py-1.5 rounded-lg border border-ink/20 hover:border-ink/40 text-xs font-medium text-ink hover:bg-ink/5 transition-colors cursor-pointer"
+            >
+              <Sparkles size={14} className="text-ink/70" />
+              <span>Browse Catalog</span>
+            </button>
+          )}
+        </div>
       </div>
 
+      {readOnly && (
+        <div className="rounded-lg border border-ink/15 bg-ink/5 px-3 py-2 text-[11px] text-ink/70 leading-relaxed">
+          This skill is provided by <span className="font-mono text-ink/80">{skill?.source}</span> and is read-only here.
+          Edit <span className="font-mono text-ink/80">{skill?.filePath}</span> at its source, or install a copy into
+          {' '}<span className="font-mono text-ink/80">~/.omp/agent/skills</span> to own it.
+        </div>
+      )}
+
       {/* Basic Information Section */}
       <div className="space-y-4">
         <h3 className="text-xs font-bold uppercase tracking-wider text-ink">
@@ -99,7 +146,8 @@ description: ""
             <div className="group relative">
               <Info size={13} className="text-ink/40 cursor-help" />
               <div className="absolute left-0 top-5 hidden group-hover:block w-64 p-2 rounded-lg bg-ink text-paper text-[11px] shadow-lg z-20 pointer-events-none leading-relaxed">
-                Unique identifier and runtime execution scope for this skill.
+                Unique identifier and runtime execution scope for this skill. A project skill is written under
+                {' '}<span className="font-mono">&lt;workspace&gt;/.omp/skills</span>.
               </div>
             </div>
           </div>
@@ -109,14 +157,42 @@ description: ""
               type="text"
               value={name}
               onChange={(e) => setName(e.currentTarget.value)}
+              disabled={readOnly}
               placeholder="skill-name"
-              className="flex-1 px-3 py-2 rounded-lg border border-ink/20 hover:border-ink/40 bg-paper focus:outline-none focus:border-ink text-xs font-mono text-ink transition-colors"
+              className="flex-1 px-3 py-2 rounded-lg border border-ink/20 hover:border-ink/40 bg-paper focus:outline-none focus:border-ink text-xs font-mono text-ink transition-colors disabled:opacity-60"
             />
             <div className="flex items-center gap-1.5 px-3 py-2 rounded-lg border border-ink/20 bg-ink/5 text-xs text-ink/80 flex-shrink-0">
               <User size={14} className="text-ink/60" />
               <span>{locationLabel}</span>
             </div>
           </div>
+
+          {/* Scope selector: only a chamber-managed skill can move roots, and
+              the project scope needs a workspace to land in. */}
+          {!readOnly && (
+            <div className="flex items-center gap-2 mt-2">
+              {(['user', 'project'] as const).map((scope) => {
+                const disabled = scope === 'project' && projectScopeDisabled;
+                return (
+                  <button
+                    key={scope}
+                    type="button"
+                    disabled={disabled}
+                    title={disabled ? 'Select a workspace folder to write a project skill' : undefined}
+                    onClick={() => {
+                      setLocation(scope);
+                      setLocationLabel(scope === 'user' ? 'User / omp agent' : 'Project / .omp/skills');
+                    }}
+                    className={`px-2.5 py-1 rounded-md text-[11px] font-medium transition-colors cursor-pointer disabled:opacity-40 disabled:cursor-not-allowed ${
+                      location === scope ? 'bg-ink text-paper' : 'border border-ink/20 text-ink/70 hover:bg-ink/5'
+                    }`}
+                  >
+                    {scope === 'user' ? 'User scope' : 'Project scope'}
+                  </button>
+                );
+              })}
+            </div>
+          )}
         </div>
 
         {/* Description */}
@@ -128,7 +204,8 @@ description: ""
             <div className="group relative">
               <Info size={13} className="text-ink/40 cursor-help" />
               <div className="absolute left-0 top-5 hidden group-hover:block w-64 p-2 rounded-lg bg-ink text-paper text-[11px] shadow-lg z-20 pointer-events-none leading-relaxed">
-                Short description helping autonomous agents decide when to invoke this skill.
+                Short description helping autonomous agents decide when to invoke this skill. Required — omp does not
+                load a SKILL.md without one.
               </div>
             </div>
           </div>
@@ -137,10 +214,29 @@ description: ""
             rows={3}
             value={description}
             onChange={(e) => setDescription(e.currentTarget.value)}
+            disabled={readOnly}
             placeholder="Brief description of what this skill does..."
-            className="w-full px-3 py-2 rounded-lg border border-ink/20 hover:border-ink/40 bg-paper focus:outline-none focus:border-ink text-xs text-ink transition-colors resize-none leading-relaxed"
+            className="w-full px-3 py-2 rounded-lg border border-ink/20 hover:border-ink/40 bg-paper focus:outline-none focus:border-ink text-xs text-ink transition-colors resize-none leading-relaxed disabled:opacity-60"
           />
         </div>
+
+        {/* Model invocation toggle */}
+        <label className="flex items-start gap-2 cursor-pointer select-none">
+          <input
+            type="checkbox"
+            checked={hidden}
+            disabled={readOnly}
+            onChange={(e) => setHidden(e.currentTarget.checked)}
+            className="mt-0.5 accent-ink cursor-pointer"
+          />
+          <span className="text-xs text-ink/80 leading-relaxed">
+            Hide from the model&apos;s skill listing
+            <span className="block text-[11px] text-ink/50">
+              The skill stays runnable via <span className="font-mono">/skill:{name || 'name'}</span>; it is only kept out
+              of the automatic skill catalog the model sees.
+            </span>
+          </span>
+        </label>
       </div>
 
       {/* Instructions Section */}
@@ -173,8 +269,9 @@ description: ""
               rows={Math.max(lineCount, 8)}
               value={instructions}
               onChange={(e) => setInstructions(e.currentTarget.value)}
+              disabled={readOnly}
               spellcheck={false}
-              className="flex-1 p-3 bg-transparent text-ink font-mono text-xs leading-relaxed resize-none focus:outline-none overflow-y-auto"
+              className="flex-1 p-3 bg-transparent text-ink font-mono text-xs leading-relaxed resize-none focus:outline-none overflow-y-auto disabled:opacity-60"
               placeholder="Type markdown instructions..."
             />
           </div>
@@ -183,7 +280,7 @@ description: ""
 
       {/* Footer Actions */}
       <div className="pt-4 border-t border-ink/10 flex items-center justify-between">
-        {!isCreatingNew && skill && onDelete ? (
+        {!isCreatingNew && skill && onDelete && !readOnly ? (
           <button
             type="button"
             onClick={() => onDelete(skill.id)}
@@ -196,14 +293,16 @@ description: ""
           <div />
         )}
 
-        <button
-          type="button"
-          onClick={handleSave}
-          className="flex items-center gap-1.5 px-4 py-2 rounded-lg bg-ink text-paper text-xs font-medium hover:bg-ink/90 transition-colors shadow-xs cursor-pointer"
-        >
-          {savedToast ? <Check size={14} /> : <Save size={14} />}
-          <span>{savedToast ? 'Saved' : isCreatingNew ? 'Create Skill' : 'Save Changes'}</span>
-        </button>
+        {!readOnly && (
+          <button
+            type="button"
+            onClick={handleSave}
+            className="flex items-center gap-1.5 px-4 py-2 rounded-lg bg-ink text-paper text-xs font-medium hover:bg-ink/90 transition-colors shadow-xs cursor-pointer"
+          >
+            {savedToast ? <Check size={14} /> : <Save size={14} />}
+            <span>{savedToast ? 'Saved' : isCreatingNew ? 'Create Skill' : 'Save Changes'}</span>
+          </button>
+        )}
       </div>
     </div>
   );

package/src/client/components/settings/categories/skill-settings/SidebarList.tsx

@@ -1,7 +1,7 @@
 import { useState } from 'preact/hooks';
 import { ChevronDown, ChevronRight, Plus } from 'lucide-preact';
 import type { SkillItem } from '@/shared/types';
-import { ProjectSelectorDropdown } from '@/client/components/settings/ProjectSelectorDropdown';
+import { ProjectSelectorDropdown, type ProjectOption } from '@/client/components/settings/ProjectSelectorDropdown';
 
 interface SkillSidebarListProps {
   skills: SkillItem[];
@@ -11,6 +11,8 @@ interface SkillSidebarListProps {
   onAddNewSkill: () => void;
   selectedProject: string;
   onSelectProject: (project: string) => void;
+  /** Real workspace folders; defaults to the shared placeholder list. */
+  projectOptions?: ProjectOption[];
 }
 
 export function SkillSidebarList({
@@ -21,6 +23,7 @@ export function SkillSidebarList({
   onAddNewSkill,
   selectedProject,
   onSelectProject,
+  projectOptions,
 }: SkillSidebarListProps) {
   const [collapsedGroups, setCollapsedGroups] = useState<Record<string, boolean>>({});
 
@@ -55,8 +58,9 @@ export function SkillSidebarList({
         variant="compact"
         selectedProject={selectedProject}
         onChangeProject={onSelectProject}
+        {...(projectOptions ? { options: projectOptions } : {})}
         getTriggerLabel={(options, value) =>
-          options.find((option) => option.id === value)?.label || value
+          options.find((option) => option.id === value || option.value === value)?.label || value
         }
       />
 
@@ -77,8 +81,10 @@ export function SkillSidebarList({
 
       {/* Skills List */}
       <div className="flex-1 scrollbar-overlay-container scrollbar-overlay-static p-1.5 space-y-2">
+        {/* The picker above names the scope; the list is exactly that scope's
+            skills, so a second scope claim here would only be able to lie. */}
         <div className="px-2 pt-1 pb-0.5 text-[10px] font-bold text-ink/50 uppercase tracking-wider">
-          USER SKILLS
+          SKILLS
         </div>
 
         {/* Temporary "New Skill" indicator item */}

package/src/client/data/settings/command.ts

@@ -7,8 +7,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'init',
     description: 'guided AGENTS.md setup',
     scope: 'system',
-    overrideAgent: 'Not selected',
-    overrideModel: 'Not selected',
     template: `Initialize AGENTS.md instructions for this repository following guided prompts. Context: $ARGUMENTS`,
     isBuiltIn: true,
   },
@@ -17,8 +15,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'review',
     description: 'review changes [commit|branch|working]',
     scope: 'system',
-    overrideAgent: 'code-reviewer',
-    overrideModel: 'Not selected',
     template: `Review the current changes in $ARGUMENTS and summarize bugs, formatting issues, and test considerations.\n!\`git diff --stat\``,
     isBuiltIn: true,
   },
@@ -28,8 +24,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'debugging',
     description: 'MUST USE for any real runtime debugging session',
     scope: 'user',
-    overrideAgent: 'coder',
-    overrideModel: 'Not selected',
     template: `Analyze the runtime error log and isolate the root failure:\n!\`bun run test\`\nFix the issue and verify: $ARGUMENTS`,
   },
   {
@@ -37,8 +31,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'frontend',
     description: 'MUST USE for frontend/web UI/UX development',
     scope: 'user',
-    overrideAgent: 'frontend-specialist',
-    overrideModel: 'Not selected',
     template: `Implement clean responsive UI component following Tailwind and theme variable specifications.\nTarget file: @$ARGUMENTS`,
   },
   {
@@ -46,8 +38,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'git-master',
     description: 'MUST USE for ANY git operations and commit conventions',
     scope: 'user',
-    overrideAgent: 'git-sync',
-    overrideModel: 'Not selected',
     template: `Inspect git status and generate clean conventional commit message:\n!\`git status -s\`\nDetails: $ARGUMENTS`,
   },
   {
@@ -55,8 +45,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'goal',
     description: '(builtin) Set, show, pause, resume session goal',
     scope: 'user',
-    overrideAgent: 'Sisyphus - ultraworker',
-    overrideModel: 'Not selected',
     template: `Manage current goal state: $ARGUMENTS`,
   },
   {
@@ -64,8 +52,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'handoff',
     description: '(builtin) Create a detailed context handoff report',
     scope: 'user',
-    overrideAgent: 'general',
-    overrideModel: 'Not selected',
     template: `Summarize work in progress, architectural decisions, and open items for handoff to next agent session.\n$ARGUMENTS`,
   },
   {
@@ -73,8 +59,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'hyperplan',
     description: '(builtin) Adversarial multi-agent plan builder',
     scope: 'user',
-    overrideAgent: 'Prometheus - Plan Builder',
-    overrideModel: 'Not selected',
     template: `Construct a detailed plan with review gates and adversarial critiques for $ARGUMENTS.`,
   },
   {
@@ -82,8 +66,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'init-deep',
     description: '(builtin) Initialize hierarchical AGENTS.md setup',
     scope: 'user',
-    overrideAgent: 'architect',
-    overrideModel: 'Not selected',
     template: `Analyze directory structure and construct domain-specific AGENTS.md files for subpackages: $ARGUMENTS`,
   },
   {
@@ -91,8 +73,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'playwright',
     description: 'MUST USE for any browser-related testing',
     scope: 'user',
-    overrideAgent: 'e2e-runner',
-    overrideModel: 'Not selected',
     template: `Run playwright tests for scenario: $ARGUMENTS\n!\`bunx playwright test\``,
   },
   {
@@ -100,8 +80,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'pty-open-background-spy',
     description: 'Open PTY Sessions Web Interface',
     scope: 'user',
-    overrideAgent: 'devops-runner',
-    overrideModel: 'Not selected',
     template: `Launch PTY background session monitor with target: $ARGUMENTS`,
   },
   {
@@ -109,8 +87,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'pty-show-server-url',
     description: 'Show PTY Sessions Web Interface URL',
     scope: 'user',
-    overrideAgent: 'devops-runner',
-    overrideModel: 'Not selected',
     template: `Output the active PTY session web socket endpoint and UI URL.`,
   },
   {
@@ -118,8 +94,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'refactor',
     description: '(builtin) Intelligent refactoring coordinator',
     scope: 'user',
-    overrideAgent: 'refactor-pro',
-    overrideModel: 'Not selected',
     template: `Refactor module @$ARGUMENTS into decoupled single-responsibility files without changing public behavior. Keep files under 350 lines.`,
   },
   {
@@ -127,8 +101,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'benchmark',
     description: 'Run performance benchmark suites and record timings',
     scope: 'user',
-    overrideAgent: 'performance-tuner',
-    overrideModel: 'Not selected',
     template: `Execute benchmark suite on target: $ARGUMENTS`,
   },
   {
@@ -136,8 +108,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'bundle-analyze',
     description: 'Inspect vendor bundle size and dynamic chunk splitting',
     scope: 'user',
-    overrideAgent: 'bundle-optimizer',
-    overrideModel: 'Not selected',
     template: `Run build analyzer and report chunk weight statistics: !\`bun run build\``,
   },
   {
@@ -145,8 +115,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'db-migrate',
     description: 'Review and execute database migrations safely',
     scope: 'user',
-    overrideAgent: 'database-analyzer',
-    overrideModel: 'Not selected',
     template: `Review pending schema migrations and verify backward compatibility: $ARGUMENTS`,
   },
   {
@@ -154,8 +122,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'doc-generate',
     description: 'Generate API documentation and usage guides',
     scope: 'user',
-    overrideAgent: 'doc-writer',
-    overrideModel: 'Not selected',
     template: `Draft comprehensive technical documentation for module @$ARGUMENTS`,
   },
   {
@@ -163,8 +129,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'security-scan',
     description: 'Run static security analysis and check dependency vulnerabilities',
     scope: 'user',
-    overrideAgent: 'security-auditor',
-    overrideModel: 'Not selected',
     template: `Scan repository for leaked secrets and outdated dependencies: !\`npm audit\``,
   },
   {
@@ -172,8 +136,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'test-unit',
     description: 'Run unit test suite with coverage report',
     scope: 'user',
-    overrideAgent: 'test-engineer',
-    overrideModel: 'Not selected',
     template: `Run test suite with coverage: !\`bun test\` - Focus: $ARGUMENTS`,
   },
   {
@@ -181,8 +143,6 @@ export const DEFAULT_COMMANDS_LIST: CommandItem[] = [
     name: 'typecheck',
     description: 'Run strict TypeScript compiler verification',
     scope: 'user',
-    overrideAgent: 'lint-fixer',
-    overrideModel: 'Not selected',
     template: `Run strict typecheck without emitting files: !\`bunx tsc --noEmit\``,
   },
 ];

package/src/client/hooks/settings/workspace-roots.ts

@@ -0,0 +1,99 @@
+/**
+ * @license
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+import { useEffect, useState } from 'preact/hooks';
+import type { ProjectOption } from '@/client/components/settings/ProjectSelectorDropdown';
+
+interface ProjectConfigItem {
+  id: string;
+  name: string;
+  path?: string;
+}
+
+export interface WorkspaceRoots {
+  options: ProjectOption[];
+  /** Workspace root for a selected option id, or null for the user scope. */
+  rootFor: (optionId: string) => string | null;
+  /** Query string (with leading `?`) for a read scoped to this option. */
+  queryFor: (optionId: string) => string;
+  /** Extra body fields scoping a write to this option. */
+  scopeBodyFor: (optionId: string) => { root?: string; scope?: string };
+  /** True when this option is a real workspace (the project scope exists). */
+  isWorkspace: (optionId: string) => boolean;
+}
+
+/** Option id for the user-level (no workspace) scope. */
+export const GLOBAL_SCOPE_ID = 'global';
+
+const USER_OPTION: ProjectOption = {
+  id: GLOBAL_SCOPE_ID,
+  name: 'global',
+  label: 'User (all projects)',
+  value: '',
+};
+
+/**
+ * The workspace folders the skills/commands panels can be scoped to.
+ *
+ * These are the real registered folders (`/api/settings/projects`), not the
+ * hardcoded placeholder list the panels used to offer: a panel scoped to a
+ * made-up project name sent a root the server could not resolve, so the picker
+ * silently did nothing. The `value` of each option is the folder's project
+ * path, which is what the API takes as `?root=`.
+ *
+ * The USER scope is `?scope=user` rather than "no root": omp resolves
+ * project-scope skills and commands from its process cwd, so an unscoped read
+ * answered for whatever directory the server happened to run in. `scope=user`
+ * pins that read to the agent dir, which is the only cwd whose inventory is
+ * the user's own.
+ */
+export function useWorkspaceRoots(): WorkspaceRoots {
+  const [options, setOptions] = useState<ProjectOption[]>([USER_OPTION]);
+
+  useEffect(() => {
+    let active = true;
+    fetch('/api/settings/projects')
+      .then((res) => res.json())
+      .then((data: { projects?: ProjectConfigItem[] }) => {
+        if (!active) return;
+        const folders = (data.projects ?? [])
+          .filter((project) => typeof project.path === 'string' && project.path)
+          .map((project) => ({
+            id: project.id,
+            name: project.name,
+            label: project.name,
+            value: project.path as string,
+          }));
+        setOptions([...folders, USER_OPTION]);
+      })
+      .catch((error) => console.error('Failed to load workspace folders:', error));
+    return () => {
+      active = false;
+    };
+  }, []);
+
+  // The dropdown reports the option's VALUE (a project path) while a panel may
+  // hold either that or the option id, so both resolve to the same root. An
+  // id-only lookup silently answered the user scope for every workspace.
+  const rootFor = (optionId: string): string | null => {
+    const option = options.find((candidate) => candidate.id === optionId || candidate.value === optionId);
+    const value = option?.value ?? '';
+    return value || null;
+  };
+
+  const queryFor = (optionId: string): string => {
+    const root = rootFor(optionId);
+    return root ? `?root=${encodeURIComponent(root)}` : '?scope=user';
+  };
+
+  const scopeBodyFor = (optionId: string): { root?: string; scope?: string } => {
+    const root = rootFor(optionId);
+    return root ? { root } : { scope: 'user' };
+  };
+
+  const isWorkspace = (optionId: string): boolean => rootFor(optionId) !== null;
+
+  return { options, rootFor, queryFor, scopeBodyFor, isWorkspace };
+}