starred/aryx
1
0
Fork 0
mirror of https://github.com/davidkaya/aryx.git synced 2026-07-25 14:08:44 +02:00
Code Issues 3 Packages Projects Releases 10 Wiki Activity

refactor: inline tool auto-approval list into tool-call approval card

Browse Source 
Move the Tool Auto-Approval Defaults section into the tool-call
ApprovalCheckpointRow card so the tool list appears directly under
the toggle instead of in a disconnected section below. Extend
ApprovalCheckpointRow with an optional children prop rendered
inside the card when the checkpoint is enabled.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
David Kaya
2026-03-25 21:51:15 +01:00
co-authored by Copilot
parent 2e3175934d
commit 08a1132d86
1 changed files with 30 additions and 30 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/renderer/components/PatternEditor.tsx
+30 -30
View File
@@ -418,7 +418,26 @@ export function PatternEditor({
onToggle={(enabled) => toggleCheckpoint('tool-call', enabled)}
scopedAgentIds={checkpointAgentIds('tool-call')}
onScopeChange={(agentIds) => setCheckpointAgentScope('tool-call', agentIds)}
/>
>
<div className="mb-1.5 text-[11px] font-semibold uppercase tracking-wider text-zinc-500">
Auto-approved tools
</div>
<p className="mb-3 text-[11px] leading-relaxed text-zinc-600">
Tools marked as auto-approved will skip manual review.
Sessions can override these defaults from the Activity panel.
</p>
{approvalTools.length === 0 ? (
<p className="py-2 text-center text-[11px] text-zinc-600">
No tools available yet. Connect MCP servers or wait for runtime capabilities to load.
</p>
) : (
<ToolApprovalGroupedList
autoApprovedSet={autoApprovedSet}
onToggle={toggleToolAutoApproval}
tools={approvalTools}
/>
)}
</ApprovalCheckpointRow>
<ApprovalCheckpointRow
agents={pattern.agents}
enabled={isCheckpointEnabled('final-response')}
@@ -431,34 +450,6 @@ export function PatternEditor({
/>
</div>
</section>
{/* Tool auto-approval */}
{isCheckpointEnabled('tool-call') && (
<section className="space-y-4">
<h4 className="text-[12px] font-semibold uppercase tracking-wider text-zinc-500">
Tool Auto-Approval Defaults
</h4>
<p className="text-[11px] leading-relaxed text-zinc-600">
Tools marked as auto-approved will skip manual review.
Sessions can override these defaults from the Activity panel.
</p>
<div className="rounded-xl border border-zinc-800 bg-zinc-900/50 px-4 py-3">
{approvalTools.length === 0 ? (
<p className="py-2 text-center text-[11px] text-zinc-600">
No tools available yet. Connect MCP servers or wait for runtime capabilities to load.
</p>
) : (
<ToolApprovalGroupedList
autoApprovedSet={autoApprovedSet}
onToggle={toggleToolAutoApproval}
tools={approvalTools}
/>
)}
</div>
</section>
)}
</div>
</div>
</div>
@@ -490,6 +481,7 @@ export function PatternEditor({
function ApprovalCheckpointRow({
agents,
children,
enabled,
kind: _kind,
label,
@@ -499,6 +491,7 @@ function ApprovalCheckpointRow({
onScopeChange,
}: {
agents: PatternAgentDefinition[];
children?: React.ReactNode;
enabled: boolean;
kind: ApprovalCheckpointKind;
label: string;
@@ -506,7 +499,7 @@ function ApprovalCheckpointRow({
onToggle: (enabled: boolean) => void;
scopedAgentIds: string[] | undefined;
onScopeChange: (agentIds: string[] | undefined) => void;
}) {
}){
const isAllAgents = !scopedAgentIds || scopedAgentIds.length === 0;
function toggleAgentScope(agentId: string) {
@@ -580,6 +573,13 @@ function ApprovalCheckpointRow({
)}
</div>
)}
{/* Optional additional content (e.g. tool auto-approval list) */}
{enabled && children && (
<div className="mt-3 border-t border-zinc-800/50 pt-3">
{children}
</div>
)}
</div>
);
}