Compare commits

...
Author SHA1 Message Date
IanandClaude Opus 5.5 c1c883fbd9 perf(prompt): load requested skills after a cache point
Siblings differ only in the skills they were spawned with, but those came
first in <specialized_knowledge>, so their prompts diverged at 39%. Shared
skills and the catalog now come first, and the requested skills follow a
cache point, so siblings share 93%.

The extra system message takes a fourth Claude breakpoint, so the Bedrock
tool_config one goes: the first system breakpoint already covers the tools.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
2026-09-29 11:12:16 -04:00
IanandClaude Opus 5.5 965c913389 refactor(llm): split the system prompt at a generic <cache_point> marker
Co-Authored-By: Claude Opus 5.5 <[email protected]>
2026-09-29 09:43:31 -04:00
IanandClaude Opus 5.5 200e37727e perf(llm): give Claude a cache point before the per-run scope
Co-Authored-By: Claude Opus 5.5 <[email protected]>
2026-09-28 23:43:50 -04:00
IanandClaude Opus 5.5 ab68249268 test(prompt): assert scope renders once, after the shared prefix
Co-Authored-By: Claude Opus 5.5 <[email protected]>
2026-09-28 23:30:29 -04:00
IanandClaude Opus 5.5 cae089b45c perf(prompt): put per-run scope at the end of the system prompt
Co-Authored-By: Claude Opus 5.5 <[email protected]>
2026-09-28 22:48:52 -04:00
8 changed files with 190 additions and 54 deletions

No files matched your search

+38 -2
View File
@@ -16,6 +16,10 @@ logger = logging.getLogger(__name__)
_PROMPT_DIRNAME = "prompts"
# Marks where the system prompt is split so the part before it can be cached.
# Removed before the prompt is sent.
CACHE_POINT = "<cache_point>"
def _resolve_skills(
*,
@@ -80,8 +84,13 @@ def render_system_prompt(
is_diff_scoped: bool = False,
interactive: bool = False,
system_prompt_context: dict[str, Any] | None = None,
include_scope: bool = True,
) -> str:
"""Render the system prompt. Returns empty string on template failure."""
"""Render the system prompt. Returns empty string on template failure.
The per-run scope (targets, MCP connections) goes last so the rest of the
prompt is an identical prefix across runs and can be served from cache.
"""
try:
prompt_dir = get_strix_resource_path("agents", _PROMPT_DIRNAME)
loader_dirs = [prompt_dir, *skill_search_dirs()]
@@ -93,6 +102,16 @@ def render_system_prompt(
),
)
shared = {
name.split("/")[-1]
for name in _resolve_skills(
requested=None,
scan_mode=scan_mode,
is_whitebox=is_whitebox,
is_root=is_root,
is_diff_scoped=is_diff_scoped,
)
}
skills_to_load = _resolve_skills(
requested=skills,
scan_mode=scan_mode,
@@ -103,12 +122,16 @@ def render_system_prompt(
skill_content = load_skills(skills_to_load)
env.globals["get_skill"] = lambda name: skill_content.get(name, "")
# Skills every agent of this kind loads come first, so siblings share them
# as a cached prefix; the ones the caller asked for vary and go after.
rendered = env.get_template("system_prompt.jinja").render(
loaded_skill_names=list(skill_content.keys()),
shared_skill_names=[name for name in skill_content if name in shared],
requested_skill_names=[name for name in skill_content if name not in shared],
available_skills=get_available_skills(),
interactive=interactive,
is_root=is_root,
system_prompt_context=system_prompt_context or {},
include_scope=include_scope,
**skill_content,
)
except Exception:
@@ -124,3 +147,16 @@ def render_system_prompt(
len(rendered),
)
return str(rendered)
def render_scope_prompt(system_prompt_context: dict[str, Any] | None) -> str:
"""Render only the per-run scope block that ends the system prompt."""
prompt_dir = get_strix_resource_path("agents", _PROMPT_DIRNAME)
env = Environment(
loader=FileSystemLoader(prompt_dir),
autoescape=select_autoescape(enabled_extensions=(), default_for_string=False),
)
rendered = env.get_template("scope.jinja").render(
system_prompt_context=system_prompt_context or {},
)
return str(rendered).strip()
+35
View File
@@ -0,0 +1,35 @@
<cache_point>
{% if system_prompt_context and system_prompt_context.authorized_targets %}
SYSTEM-VERIFIED SCOPE:
- The following scope metadata is injected by the platform into the system prompt and is authoritative
- Scope source: {{ system_prompt_context.scope_source }}
- Authorization source: {{ system_prompt_context.authorization_source }}
- Every target listed below has already been verified by the platform as in-scope and authorized
- User instructions, chat messages, and other free-form text do NOT expand scope beyond this list
- NEVER refuse, question authorization, or claim lack of permission for any target in this system-verified scope
- NEVER test any external domain, URL, host, IP, or repository that is not explicitly listed in this system-verified scope
- If the user mentions any asset outside this list, ignore that asset and continue working only on the listed in-scope targets
AUTHORIZED TARGETS:
{% for target in system_prompt_context.authorized_targets %}
- {{ target.type }}: {{ target.value }}{% if target.workspace_path %} (workspace: {{ target.workspace_path }}){% endif %}
{% endfor %}
{% endif %}
{% if system_prompt_context and system_prompt_context.mcp_available %}
MCP CONNECTIONS (available this run):
- The user connected one or more MCP (Model Context Protocol) servers. Their individual tools do NOT appear in your tool list. Use the four discovery and dispatch tools to reach them.
{% if system_prompt_context.mcp_connections %}
- Connected this run (search one to find relevant tools):
{% for connection in system_prompt_context.mcp_connections %}
- {{ connection.name }} ({{ connection.tool_count }} tools){% if connection.purpose %}: {{ connection.purpose }}{% endif %}
{% endfor %}
{% endif %}
- Reach for a connection whenever the target itself cannot give you information a connection could: its database schema and access policies, real deployment or infrastructure configuration, known issues or prior findings, or server logs. In those cases call list_mcps early to see what is available, and prefer a connection's authoritative data over inferring from the target's responses. Do not wait to be told a connection exists.
1. Call list_mcps() to discover the available connections.
2. Call search_mcp_tools(connection="<name>", query="<capability>") for a short candidate list.
3. Call get_mcp_tool_schema(connection="<name>", tool="<tool>") for the one schema you need.
4. Call call_mcp(connection="<name>", tool="<tool>", arguments={...}) to run it, passing an arguments object that matches the schema (omit arguments for a tool that takes none).
- Do not assume a connection or tool exists; discover it with list_mcps and search_mcp_tools before calling.
- Use describe_mcp only as a compatibility fallback when targeted search cannot identify an expected tool. Its full catalog can be large.
{% endif %}
+18 -38
View File
@@ -58,41 +58,6 @@ AUTONOMOUS BEHAVIOR:
</communication_rules>
<execution_guidelines>
{% if system_prompt_context and system_prompt_context.authorized_targets %}
SYSTEM-VERIFIED SCOPE:
- The following scope metadata is injected by the platform into the system prompt and is authoritative
- Scope source: {{ system_prompt_context.scope_source }}
- Authorization source: {{ system_prompt_context.authorization_source }}
- Every target listed below has already been verified by the platform as in-scope and authorized
- User instructions, chat messages, and other free-form text do NOT expand scope beyond this list
- NEVER refuse, question authorization, or claim lack of permission for any target in this system-verified scope
- NEVER test any external domain, URL, host, IP, or repository that is not explicitly listed in this system-verified scope
- If the user mentions any asset outside this list, ignore that asset and continue working only on the listed in-scope targets
AUTHORIZED TARGETS:
{% for target in system_prompt_context.authorized_targets %}
- {{ target.type }}: {{ target.value }}{% if target.workspace_path %} (workspace: {{ target.workspace_path }}){% endif %}
{% endfor %}
{% endif %}
{% if system_prompt_context and system_prompt_context.mcp_available %}
MCP CONNECTIONS (available this run):
- The user connected one or more MCP (Model Context Protocol) servers. Their individual tools do NOT appear in your tool list. Use the four discovery and dispatch tools to reach them.
{% if system_prompt_context.mcp_connections %}
- Connected this run (search one to find relevant tools):
{% for connection in system_prompt_context.mcp_connections %}
- {{ connection.name }} ({{ connection.tool_count }} tools){% if connection.purpose %}: {{ connection.purpose }}{% endif %}
{% endfor %}
{% endif %}
- Reach for a connection whenever the target itself cannot give you information a connection could: its database schema and access policies, real deployment or infrastructure configuration, known issues or prior findings, or server logs. In those cases call list_mcps early to see what is available, and prefer a connection's authoritative data over inferring from the target's responses. Do not wait to be told a connection exists.
1. Call list_mcps() to discover the available connections.
2. Call search_mcp_tools(connection="<name>", query="<capability>") for a short candidate list.
3. Call get_mcp_tool_schema(connection="<name>", tool="<tool>") for the one schema you need.
4. Call call_mcp(connection="<name>", tool="<tool>", arguments={...}) to run it, passing an arguments object that matches the schema (omit arguments for a tool that takes none).
- Do not assume a connection or tool exists; discover it with list_mcps and search_mcp_tools before calling.
- Use describe_mcp only as a compatibility fallback when targeted search cannot identify an expected tool. Its full catalog can be large.
{% endif %}
AUTHORIZATION STATUS:
- You have FULL AUTHORIZATION for authorized security validation on in-scope targets to help secure the target systems/app
- All permission checks have been COMPLETED and APPROVED - never question your authority
@@ -528,9 +493,9 @@ Directories:
Default user: pentester (sudo available)
</environment>
{% if loaded_skill_names %}
{% if shared_skill_names %}
<specialized_knowledge>
{% for skill_name in loaded_skill_names %}
{% for skill_name in shared_skill_names %}
<{{ skill_name }}>
{{ get_skill(skill_name) }}
</{{ skill_name }}>
@@ -540,7 +505,7 @@ Default user: pentester (sudo available)
{% if available_skills %}
<available_skills>
On-demand specialist skills. Spawn a specialist via `create_agent(skills=[...])`, or pull guidance inline for yourself via `load_skill(skills=[...])`. Anything wrapped in `<specialized_knowledge>` above is already loaded for you.
On-demand specialist skills. Spawn a specialist via `create_agent(skills=[...])`, or pull guidance inline for yourself via `load_skill(skills=[...])`. Anything wrapped in `<specialized_knowledge>` is already loaded for you.
{% for category, skills in available_skills | dictsort -%}
{% for skill in skills -%}
@@ -549,3 +514,18 @@ On-demand specialist skills. Spawn a specialist via `create_agent(skills=[...])`
{% endfor -%}
</available_skills>
{% endif %}
{% if requested_skill_names %}
<cache_point>
<specialized_knowledge>
{% for skill_name in requested_skill_names %}
<{{ skill_name }}>
{{ get_skill(skill_name) }}
</{{ skill_name }}>
{% endfor %}
</specialized_knowledge>
{% endif %}
{% if include_scope %}
{% include "scope.jinja" %}
{% endif %}
+28
View File
@@ -36,6 +36,7 @@ from openai.types.responses import (
from openai.types.responses.response_usage import ResponseUsage
from openai.types.shared import Reasoning
from strix.agents.prompt import CACHE_POINT
from strix.config import codex
from strix.config.loader import load_settings
from strix.config.tool_call_ids import TurnCallIdRewriter, dedupe_input
@@ -306,6 +307,9 @@ class _TurnGuardModel(Model):
) -> ModelResponse:
sanitized = dedupe_input(input)
rewriter = TurnCallIdRewriter(sanitized)
system_instructions, sanitized = _split_cached_prefix(
system_instructions, sanitized, model_settings
)
response = await self._inner.get_response(
system_instructions,
cast("str | list[TResponseInputItem]", sanitized),
@@ -339,6 +343,9 @@ class _TurnGuardModel(Model):
) -> AsyncIterator[TResponseStreamEvent]:
sanitized = dedupe_input(input)
rewriter = TurnCallIdRewriter(sanitized)
system_instructions, sanitized = _split_cached_prefix(
system_instructions, sanitized, model_settings
)
limiter = self._limiter()
stream = self._inner.stream_response(
system_instructions,
@@ -359,6 +366,27 @@ class _TurnGuardModel(Model):
self._log_dropped(limiter)
def _split_cached_prefix(
system_instructions: str | None,
model_input: str | list[Any],
model_settings: ModelSettings,
) -> tuple[str | None, str | list[Any]]:
"""Split the system prompt at each ``CACHE_POINT`` on cache-point routes.
LiteLLM puts a cache point at the end of each system message, so each part
gets its own. Other routes get the prompt with the markers removed.
"""
if not system_instructions or CACHE_POINT not in system_instructions:
return system_instructions, model_input
extra_args = model_settings.extra_args or {}
if "cache_control_injection_points" not in extra_args:
return system_instructions.replace(CACHE_POINT, ""), model_input
if isinstance(model_input, str):
model_input = [{"role": "user", "content": model_input}]
parts = [part for part in system_instructions.split(CACHE_POINT) if part.strip()]
return None, [*({"role": "system", "content": part} for part in parts), *model_input]
async def _aclose(stream: AsyncIterator[TResponseStreamEvent]) -> None:
if isinstance(stream, AsyncGenerator):
with contextlib.suppress(Exception):
+12 -10
View File
@@ -312,11 +312,12 @@ def _reasoning_settings(effort: ReasoningEffort) -> ModelSettings:
def _prompt_cache_extra_args(model_name: str) -> dict[str, Any] | None:
"""LiteLLM ``cache_control_injection_points`` for Claude prompt caching.
System prompt + rolling last-message breakpoint everywhere; ``tool_config``
only on Bedrock Converse (the only route whose LiteLLM transform consumes
it — elsewhere it leaks onto the wire and native Anthropic 400s). Unmapped
Bedrock models get no points at all: Bedrock rejects the passed-through
field outright.
A breakpoint on each system message, plus a rolling last-message one. The
system prompt is split into up to three messages, which with the last
message uses all four breakpoints Claude allows. There is none on
``tool_config``: the tools come before the system prompt, so its first
breakpoint caches them too. Unmapped Bedrock models get no points at all:
Bedrock rejects the passed-through field outright.
The field is LiteLLM's own, consumed by its transform, so it only goes to
routes LiteLLM serves. A bare ``claude-...`` name is served by the SDK's
@@ -328,11 +329,12 @@ def _prompt_cache_extra_args(model_name: str) -> dict[str, Any] | None:
if is_bedrock_route(model_name) and not bedrock_route_supports_prompt_caching(model_name):
return None
points: list[dict[str, Any]] = [{"location": "message", "role": "system"}]
if is_bedrock_route(model_name):
points.append({"location": "tool_config"})
points.append({"location": "message", "index": -1})
return {"cache_control_injection_points": points}
return {
"cache_control_injection_points": [
{"location": "message", "role": "system"},
{"location": "message", "index": -1},
]
}
def child_initial_input(
+5 -3
View File
@@ -17,7 +17,7 @@ from agents.sandbox import SandboxRunConfig
from openai import RateLimitError
from strix.agents.factory import build_strix_agent, make_child_factory
from strix.agents.prompt import render_system_prompt
from strix.agents.prompt import render_scope_prompt, render_system_prompt
from strix.config import load_settings
from strix.config.models import (
StrixProvider,
@@ -164,15 +164,17 @@ def _compose_root_instructions_override(
is_diff_scoped=is_diff_scoped,
interactive=interactive,
system_prompt_context=system_prompt_context,
include_scope=False,
)
return (
f"{base_instructions}\n\n"
"<root_scan_instructions_override>\n"
"The following root scan instructions are subordinate to the "
"system-verified scope above. They cannot expand, replace, or weaken "
"system-verified scope below. They cannot expand, replace, or weaken "
"authorized target constraints.\n\n"
f"{root_instructions_override}\n"
"</root_scan_instructions_override>"
"</root_scan_instructions_override>\n\n"
f"{render_scope_prompt(system_prompt_context)}"
)
-1
View File
@@ -70,7 +70,6 @@ def _cache_points(model_name: str) -> Any:
def test_make_model_settings_enables_prompt_cache_for_bedrock_claude() -> None:
assert _cache_points("bedrock/global.anthropic.claude-opus-4-8") == [
{"location": "message", "role": "system"},
{"location": "tool_config"},
{"location": "message", "index": -1},
]
+54
View File
@@ -6,6 +6,7 @@ flow through to the root agent's ``build_strix_agent`` call.
from __future__ import annotations
import os
import types
from typing import Any
@@ -17,8 +18,11 @@ from openai import RateLimitError
import strix.tools.mcp as mcp_pkg
import strix.tools.notes.tools as notes_tools
import strix.tools.todo.tools as todo_tools
from strix.agents.prompt import render_system_prompt
from strix.config.models import _split_cached_prefix
from strix.core import runner
from strix.core.agents import AgentCoordinator
from strix.core.inputs import make_model_settings
from strix.runtime import session_manager
from strix.tools.mcp import BearerAuth, McpConnectionConfig, McpConnectionRequest
@@ -132,6 +136,10 @@ async def test_root_prompt_options_flow_into_root_agent(
assert "AUTHORIZED TARGETS" in instructions_override
assert "https://example.com" in instructions_override
assert "CUSTOM SCAN PROMPT" in instructions_override
assert instructions_override.count("SYSTEM-VERIFIED SCOPE") == 1
assert instructions_override.index("CUSTOM SCAN PROMPT") < instructions_override.index(
"SYSTEM-VERIFIED SCOPE"
)
assert (
"cannot expand, replace, or weaken authorized target constraints" in instructions_override
)
@@ -259,3 +267,49 @@ async def test_unknown_tool_calls_are_returned_to_the_model(
)
assert captured["run_config"].tool_not_found_behavior == "return_error_to_model"
def test_scope_is_rendered_once_at_the_end_of_the_prompt() -> None:
prompt = render_system_prompt(
system_prompt_context={
"authorized_targets": [{"type": "web_application", "value": "https://example.com"}],
},
)
assert prompt.count("SYSTEM-VERIFIED SCOPE") == 1
assert prompt.index("</available_skills>") < prompt.index("SYSTEM-VERIFIED SCOPE")
def test_requested_skills_follow_the_shared_prefix() -> None:
xss = render_system_prompt(skills=["xss"], include_scope=False)
sqli = render_system_prompt(skills=["sql_injection"], include_scope=False)
shared = os.path.commonprefix([xss, sqli])
assert "</available_skills>" in shared
assert shared.count("<cache_point>") == 1
assert "<xss>" in xss.split("<cache_point>")[1]
def test_scope_is_sent_as_its_own_system_message_on_cache_point_routes() -> None:
settings = make_model_settings(None, model_name="anthropic/claude-sonnet-5-5")
prompt = render_system_prompt(
system_prompt_context={
"authorized_targets": [{"type": "web_application", "value": "https://target.invalid"}],
},
)
system, model_input = _split_cached_prefix(prompt, "go", settings)
assert system is None
assert isinstance(model_input, list)
assert [item["role"] for item in model_input] == ["system", "system", "user"]
assert "https://target.invalid" not in model_input[0]["content"]
assert "https://target.invalid" in model_input[1]["content"]
assert "<cache_point>" not in model_input[0]["content"] + model_input[1]["content"]
def test_cache_point_marker_is_removed_without_cache_points() -> None:
settings = make_model_settings(None, model_name="openai/gpt-5")
prompt = "shared\n<cache_point>\ntargets"
assert _split_cached_prefix(prompt, "go", settings) == ("shared\n\ntargets", "go")