python-agent-harness 1.5.6.5__tar.gz → 1.5.6.6__tar.gz

{python_agent_harness-1.5.6.5/python_agent_harness.egg-info → python_agent_harness-1.5.6.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-agent-harness
-Version: 1.5.6.5
+Version: 1.5.6.6
 Summary: A lightweight, hackable mini-OpenCode written in Python.
 Author-email: Huming Chen <chenhuming@gmail.com>
 License-Expression: MIT

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "python-agent-harness"
-version = "1.5.6.5"
+version = "1.5.6.6"
 description = "A lightweight, hackable mini-OpenCode written in Python."
 authors = [
     {name = "Huming Chen", email = "chenhuming@gmail.com"}

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/python_agent_harness/__init__.py

@@ -7,7 +7,7 @@ from .mcp.config import MCPConfig, MCPServerConfig
 from .mcp.manager import MCPManager
 from .session.session import Session
 
-__version__ = "1.5.6.5"
+__version__ = "1.5.6.6"
 
 __all__ = [
     "Client",

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/python_agent_harness/core/context_manager.py

@@ -65,10 +65,11 @@ class ContextManager:
     def compact(self) -> bool:
         """Compact the conversation; return True on success.
 
-        On success the history is replaced by the summary frame followed
-        by every real user prompt (nudges and other harness-injected
-        messages excluded), so the model keeps the actual requests; the
-        last prompt is the resume request for the next round.
+        On success the history is replaced by the summary frame, every
+        real user prompt (nudges and other harness-injected messages
+        excluded), and a trailing resume message telling the model what
+        is already done — the summary and the prompts alone end on a
+        bare user request, which invites redoing finished work.
         """
         loop = self.loop
         prompts = user_prompt_texts(loop.messages)
@@ -87,8 +88,10 @@ class ContextManager:
             # stays untouched): it is part of the user turn, never a
             # system message.  Every real user prompt (nudges and other
             # harness-injected messages excluded) is preserved verbatim
-            # after the frame, so the model keeps the actual requests.
-            loop.messages = compacted_messages(summary, prompts)
+            # after the frame, so the model keeps the actual requests,
+            # and the trailing resume message states that the summarised
+            # work is done (injected flag: TUI and filters skip it).
+            loop.messages = compacted_messages(summary, prompts, resume=True)
             # The shared conversation now is the compacted one: mirror it
             # onto session.last_messages so the TUI (renders from it) and
             # a later manual /compact start from the summary, not the old

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/python_agent_harness/prompts/agent.md

@@ -69,21 +69,6 @@ The user will primarily request you perform software engineering tasks. This inc
 - Batch independent tool calls in a single response to maximize parallelism.
 
 # Tool usage instructions
-**Specialized Tools vs. Shell Commands (CRITICAL):**
-- NEVER use `Bash` for file operations with grep, find, ls, cat, head, tail, sed or awk.
-- ALWAYS use: `Glob`, `Grep`, `Read`, `Edit`, `Write`
-- Reserve `Bash` EXCLUSIVELY for: git, npm, docker, cargo, make, system services and other non-file commands
-- Using bash for file operations violates the tool hierarchy and creates technical debt
-
-**Tool Selection Hierarchy:**
-- File search by name → Use `Glob` (NOT find or ls)
-- Directory listing → Use `Glob` with glob pattern `"*"` (not ls)
-- Content search → Use `Grep` (NOT grep or rg)
-- Read files → Use `Read` (NOT cat/head/tail)
-- Edit files → Use `Edit` (NOT sed/awk)
-- Write files → Use `Write` (NOT echo >/cat <<EOF)
-- System operations → Use `Bash` (for git, npm, docker, etc.)
-
 {{TOOL_INSTRUCTIONS}}
 
 # Code References

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/python_agent_harness/prompts/core.py

@@ -500,7 +500,11 @@ def user_prompt_texts(messages: list) -> list[str]:
     (``remember_user_text`` excludes reminders for title generation —
     a different purpose, so the filters differ.)
     """
-    nudge = config.NUDGE_MESSAGE
+    # Harness-injected one-liners replayed verbatim on every compaction
+    # would accumulate: the completion nudge, and the resume message the
+    # automatic compaction appends (a bare "keep going" carries no task
+    # content — the summary already covers the progress it refers to).
+    injected = {config.NUDGE_MESSAGE, config.RESUME_MESSAGE}
     is_reminder: list[bool] = []
     last_reminder = -1
     for msg in messages:
@@ -523,7 +527,7 @@ def user_prompt_texts(messages: list) -> list[str]:
         if _message_role(msg) != "user":
             continue
         text = _message_text(msg)
-        if not text or text == nudge:
+        if not text or text in injected:
             continue
         if text.startswith(config.COMPACT_HEADER):
             continue
@@ -587,19 +591,29 @@ def compact_summary(
     return summary or None
 
 
-def compacted_messages(summary: str, prompts: list[str]) -> list[Message]:
+def compacted_messages(summary: str, prompts: list[str], resume: bool = False) -> list[Message]:
     """The post-compaction history: the summary frame as a user message
     followed by every preserved user prompt, oldest first.
 
     The summary lives in the user turn (the system prompt is passed
     separately and stays untouched); *prompts* are the real user
     requests (see ``user_prompt_texts``) that must survive compaction.
+
+    *resume* appends ``config.RESUME_MESSAGE`` (flagged ``injected``, so
+    the TUI and the filters skip it) after the prompts: without it the
+    rebuilt history ends with a bare user request and the model may redo
+    work the summary already records.  Only the automatic in-loop
+    compaction passes it — the manual /compact command waits for the
+    next user message instead.
     """
     frame = (config.COMPACT_HEADER + summary + config.COMPACT_SEPARATOR).strip()
-    return [
+    messages = [
         Message(role="user", content=frame),
         *[Message(role="user", content=p) for p in prompts],
     ]
+    if resume:
+        messages.append(Message(role="user", content=config.RESUME_MESSAGE, injected=True))
+    return messages
 
 
 # ---- custom agent discovery -----------------------------------------------

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/python_agent_harness/prompts/subagent.md

@@ -20,26 +20,12 @@ You are an autonomous subagent. Your role is to independently complete well-defi
 You MUST answer concisely with fewer than 4 lines of text (not including tool use or code generation), unless user asks for detail.
 
 # Tool usage instructions
-**Specialized Tools vs. Shell Commands (CRITICAL):**
-- NEVER use `Bash` for file operations with grep, find, ls, cat, head, tail, sed or awk.
-- ALWAYS use: `Glob`, `Grep`, `Read`, `Edit`, `Write`
-- Reserve `Bash` EXCLUSIVELY for: git, npm, docker, cargo, make, system services and other non-file commands
-- Using bash for file operations violates the tool hierarchy and creates technical debt
 
 **Parallel Tool Execution:**
 - Call multiple tools in a single response when tasks are independent
 - Never use placeholders or guess missing parameters
 - Maximize parallel execution to improve efficiency
 
-**Tool Selection Hierarchy:**
-- File search by name → Use `Glob` (NOT find or ls)
-- Directory listing → Use `Glob` with glob pattern `"*"` (not ls)
-- Content search → Use `Grep` (NOT grep or rg)
-- Read files → Use `Read` (NOT cat/head/tail)
-- Edit files → Use `Edit` (NOT sed/awk)
-- Write files → Use `Write` (NOT echo >/cat <<EOF)
-- System operations → Use `Bash` (for git, npm, docker, etc.)
-
 {{TOOL_INSTRUCTIONS}}
 
 # Output Requirements

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/python_agent_harness/session/config.py

@@ -44,6 +44,18 @@ NUDGE_MESSAGE = (
 # ---- compaction -----------------------------------------------------------
 COMPACT_HEADER = "**[Compacted Summary]**\n\n"
 COMPACT_SEPARATOR = "\n\n---\n\n**[Context compacted]**\n\n---\n\n"
+# Appended after the summary frame + preserved prompts on the AUTOMATIC
+# path only (AgentLoop -> ContextManager.compact).  The rebuilt history
+# ends with the original user request, and a bare request reads as "start
+# here", so the model tends to redo work the summary already records.  The
+# manual /compact command does NOT inject it: it replaces the history and
+# waits for the next user message.
+RESUME_MESSAGE = (
+    "Compaction just finished; the summary above records the work already done. "
+    "Do not repeat completed steps or re-run finished tool calls. "
+    "Review which parts of the request above are already done "
+    "and continue with the unfinished ones."
+)
 
 # ---- token calibration ----------------------------------------------------
 CALIBRATION_MIN = 0.5

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/python_agent_harness/session/session.py

@@ -670,14 +670,16 @@ class Session:
     def remember_user_text(self, messages: list) -> None:
         """Remember the last real user message for session-title generation.
 
-        Skips harness-injected messages (nudges, plan/build-switch
-        reminders, queued mode prompts — flagged ``injected``) so a
-        title is never generated from "Review the original user request
-        and the Task Completion Rules…" or a mode-switch reminder.
+        Skips harness-injected messages (nudges, the post-compaction
+        resume message, plan/build-switch reminders, queued mode prompts
+        — flagged ``injected``; the content check also covers restored
+        sessions, where the flag is lost) so a title is never generated
+        from "Review the original user request and the Task Completion
+        Rules…" or a mode-switch reminder.
         """
-        nudge = config.NUDGE_MESSAGE
+        injected = {config.NUDGE_MESSAGE, config.RESUME_MESSAGE}
         for m in reversed(messages):
-            if m.role == "user" and not m.injected and m.text() != nudge:
+            if m.role == "user" and not m.injected and m.text() not in injected:
                 self.store.remember_first_user_message(m.text())
                 break
 

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6}/python_agent_harness/tui/render.py

@@ -144,11 +144,13 @@ def _is_injected_user_text(text: str) -> bool:
 
     Live sessions flag injected messages directly; these content checks
     cover restored sessions, where the flag is lost in the markdown
-    round-trip: the completion nudge, the <system-reminder>-wrapped
-    plan/build-switch prompts (plan.md, plan-mode.md, build-switch.md
-    all start with the reminder tag), and the plan-exit approval notice.
+    round-trip: the completion nudge, the post-compaction resume message,
+    the <system-reminder>-wrapped plan/build-switch prompts (plan.md,
+    plan-mode.md, build-switch.md all start with the reminder tag), and
+    the plan-exit approval notice.
     """
-    if text == config.NUDGE_MESSAGE:
+    injected = {config.NUDGE_MESSAGE, config.RESUME_MESSAGE}
+    if text in injected:
         return True
     return _is_mode_reminder_text(text)
 

{python_agent_harness-1.5.6.5 → python_agent_harness-1.5.6.6/python_agent_harness.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-agent-harness
-Version: 1.5.6.5
+Version: 1.5.6.6
 Summary: A lightweight, hackable mini-OpenCode written in Python.
 Author-email: Huming Chen <chenhuming@gmail.com>
 License-Expression: MIT