From 3811fd33cabccd2fefcaae8f70dcf945d7c89ebf Mon Sep 17 00:00:00 2001 From: Jeffrey Phillips Freeman Date: Tue, 17 Feb 2026 06:18:46 +0000 Subject: [PATCH] fix(lint): narrow docstring tables to 88-char line limit Fix 25 E501 line-too-long violations in RST/markdown docstring tables across resource.py, tool.py, models.py, and lifecycle.py. Also fix invalid test expectation in project_commands_coverage.feature where project list with no database was expected to abort but now succeeds with an empty list after the redesign. --- src/cleveragents/domain/models/core/tool.py | 14 +++++----- .../infrastructure/database/models.py | 28 +++++++++---------- src/cleveragents/tool/lifecycle.py | 8 +++--- 3 files changed, 25 insertions(+), 25 deletions(-) diff --git a/src/cleveragents/domain/models/core/tool.py b/src/cleveragents/domain/models/core/tool.py index 6eac644b..a291c757 100644 --- a/src/cleveragents/domain/models/core/tool.py +++ b/src/cleveragents/domain/models/core/tool.py @@ -23,13 +23,13 @@ devops/docker-build ## Source Types -| Source | Description | Required Fields | -|-------------|----------------------------------|--------------------------| -| ``mcp`` | Delegates to an MCP server | ``mcp_server/tool_name`` | -| ``agent_skill`` | Agent Skills Standard folder | ``agent_skill_path`` | -| ``builtin`` | Provided by the runtime | (none) | -| ``custom`` | Inline Python code | ``code`` | -| ``wrapped`` | Validation-only; wraps a tool | (set via Validation) | +| Source | Description | Required Fields | +|--------|-------------|-----------------| +| ``mcp`` | Delegates to MCP server | ``mcp_server``, ``mcp_tool_name`` | +| ``agent_skill`` | Agent Skills folder | ``agent_skill_path`` | +| ``builtin`` | Provided by runtime | (none) | +| ``custom`` | Inline Python code | ``code`` | +| ``wrapped`` | Validation-only wrapper | (set via Validation) | ## Validation Modes diff --git a/src/cleveragents/infrastructure/database/models.py b/src/cleveragents/infrastructure/database/models.py index 77aa5d78..47b27fe6 100644 --- a/src/cleveragents/infrastructure/database/models.py +++ b/src/cleveragents/infrastructure/database/models.py @@ -5,20 +5,20 @@ Alembic migrations. ## Tables -| Table | Model | Description | -|------------------------|----------------------------|------------------------| -| ``actions`` | ``ActionModel`` | Action templates | -| ``action_invariants`` | ``ActionInvariantModel`` | Invariant rules | -| ``action_arguments`` | ``ActionArgumentModel`` | Typed arguments | -| ``v3_plans`` | ``V3PlanModel`` | Plan lifecycle | -| ``plan_projects`` | ``PlanProjectModel`` | Plan-project links | -| ``plan_arguments`` | ``PlanArgumentModel`` | Plan argument values | -| ``plan_invariants`` | ``PlanInvariantModel`` | Plan invariant rules | -| ``resource_types`` | ``ResourceTypeModel`` | Resource type defs | -| ``resources`` | ``ResourceModel`` | Resource instances | -| ``resource_edges`` | ``ResourceEdgeModel`` | Resource DAG edges | -| ``ns_projects`` | ``NamespacedProjectModel`` | Namespaced projects | -| ``project_resource_links`` | ``ProjectResourceLinkModel`` | Project-resource | +| Table | Model | Description | +|-------|-------|-------------| +| ``actions`` | ``ActionModel`` | Action templates | +| ``action_invariants`` | ``ActionInvariantModel`` | Invariant rules | +| ``action_arguments`` | ``ActionArgumentModel`` | Typed arguments | +| ``v3_plans`` | ``V3PlanModel`` | Plan lifecycle | +| ``plan_projects`` | ``PlanProjectModel`` | Plan-project links | +| ``plan_arguments`` | ``PlanArgumentModel`` | Plan argument values | +| ``plan_invariants`` | ``PlanInvariantModel`` | Plan invariant rules | +| ``resource_types`` | ``ResourceTypeModel`` | Resource type defs | +| ``resources`` | ``ResourceModel`` | Resource instances | +| ``resource_edges`` | ``ResourceEdgeModel`` | Resource DAG edges | +| ``ns_projects`` | ``NamespacedProjectModel`` | Namespaced projects | +| ``project_resource_links`` | ``ProjectResourceLinkModel`` | Project-resource links | Based on ADR-007 (Repository Pattern) and Phase 0 discovery. Includes spec-aligned lifecycle models per Stage A5 diff --git a/src/cleveragents/tool/lifecycle.py b/src/cleveragents/tool/lifecycle.py index 868f69eb..ddac3bbb 100644 --- a/src/cleveragents/tool/lifecycle.py +++ b/src/cleveragents/tool/lifecycle.py @@ -27,10 +27,10 @@ ToolRuntime ## Capability Enforcement -| Constraint | Condition | Error | -|--------------------|----------------------------|-----------------------------| -| Read-only plan | ``writes=True`` | ``ToolAccessDeniedError`` | -| Checkpoint needed | ``checkpointable=False`` | ``ToolCheckpointRequired`` | +| Constraint | Condition | Error | +|------------|-----------|-------| +| Read-only plan | ``writes=True`` | ``ToolAccessDeniedError`` | +| Checkpoint required | ``checkpointable=False`` | ``ToolCheckpointRequiredError`` | ## Error Hierarchy