backend-refactor

Category: Development & Coding | Uploader: Qredence | Downloads: 0 | Version: v1.0（Latest）
Simplifies fleet-rlm backend by rewriting/deleting modules and updating tests
Changelog: Source: GitHub https://github.com/Qredence/fleet-rlm
Directory Structure

Current level: tree/main/.factory/skills/backend-refactor/
📄 SKILL.md 4.6 KB
SKILL.md

---
name: backend-refactor
description: Simplifies fleet-rlm backend by rewriting/deleting modules and updating tests
---

# Backend Refactor Worker

NOTE: Startup and cleanup are handled by `worker-base`. This skill defines the WORK PROCEDURE.

## When to Use This Skill

Any feature that involves:
- Rewriting Python modules in `src/fleet_rlm/runtime/`, `src/fleet_rlm/api/`, `src/fleet_rlm/integrations/daytona/`, or `src/fleet_rlm/integrations/database/`
- Deleting directories or orphaned files
- Creating new simplified modules
- Updating or creating unit tests
- Removing dangling imports
- Merging multiple modules into one
- Upgrading dependencies with breaking changes

## Required Skills

None.

## Work Procedure

1. **Read the feature description** carefully. Understand exactly what files to create, modify, or delete.

2. **Read existing code first.** Before modifying any file, read it to understand current patterns, imports, and conventions. Read `.factory/library/architecture.md` and `.factory/library/environment.md` for context.

3. **Write tests first** (TDD). For new modules, write failing tests in `tests/unit/runtime/` before implementation. For deletions, skip this step.

4. **Implement the change.** Write clean, minimal code following existing conventions:
   - Type hints on all signatures
   - No import-time side effects
   - Follow ruff formatting
   - DSPy 3.1.3: tools are plain callables or `dspy.Tool(func)`. Use `@tool_fn` marker from `runtime/tools/_marker.py` for registry discovery.
   
5. **For deletion features:** Delete the directories/files, then grep for dangling imports and fix them. Remove corresponding test directories. Also verify no dynamic/runtime imports are broken (e.g., `discover_tools()` scanning `runtime/tools/*.py`).

6. **For consolidation features:** Before merging files, read ALL files involved. Merge into the most natural target file. Update all imports across the codebase. After merging, delete the old files and verify no dangling imports.

7. **For Daytona 0.168.0 upgrade:** Search for camelCase attribute access on Daytona objects (e.g., `autoStopInterval`, `autoArchiveInterval`, `idleTimeout`). Daytona 0.168.0 enforces snake_case. Fix all occurrences to snake_case. Run `uv sync --all-extras` after updating `pyproject.toml`.

8. **Run validation:**
   ```bash
   make format
   make lint
   make typecheck
   make test-unit
   ```
   Fix all failures before completing.

9. **Verify no dangling imports** for any deleted or consolidated modules:
   ```bash
   rg "from fleet_rlm\.(agent_host|worker)" src/fleet_rlm/
   rg "import fleet_rlm\.(agent_host|worker)" src/fleet_rlm/
   ```
   Also search for imports of the specific deleted module names.

## Example Handoff

```json
{
  "salientSummary": "Created simplified dspy.ReAct agent module in runtime/agent/agent.py with RLMReActChatSignature. Wrote 6 unit tests covering module construction, forward(), optimizer compat, and tool registration. All pass. make lint + make typecheck clean.",
  "whatWasImplemented": "New runtime/agent/agent.py with FleetAgent(dspy.Module) wrapping dspy.ReAct. Constructor accepts tools list and max_iters. Forward passes chat_history and user_message through ReAct. Tests in tests/unit/runtime/agent/test_agent.py.",
  "whatWasLeftUndone": "",
  "verification": {
    "commandsRun": [
      { "command": "make format", "exitCode": 0, "observation": "No changes needed" },
      { "command": "make lint", "exitCode": 0, "observation": "Clean" },
      { "command": "make typecheck", "exitCode": 0, "observation": "Clean" },
      { "command": "make test", "exitCode": 0, "observation": "142 passed, 3 skipped" },
      { "command": "rg 'from fleet_rlm.agent_host' src/fleet_rlm/", "exitCode": 1, "observation": "No matches" }
    ],
    "interactiveChecks": []
  },
  "tests": {
    "added": [
      {
        "file": "tests/unit/runtime/agent/test_agent.py",
        "cases": [
          { "name": "test_agent_is_dspy_module", "verifies": "Agent subclasses dspy.Module" },
          { "name": "test_agent_forward_returns_prediction", "verifies": "forward() returns dspy.Prediction" },
          { "name": "test_agent_accepts_tools", "verifies": "Constructor accepts tools list" }
        ]
      }
    ]
  },
  "discoveredIssues": []
}
```

## When to Return to Orchestrator

- A module you need to modify is not in the expected location or has unexpected structure
- Existing tests fail before your changes (pre-existing failures)
- DSPy API behaves differently than documented in `.factory/library/environment.md`
- You need to modify off-limits directories (observability/, mcp/, config/, quality/, frontend/, scaffold/)
Comments 0
Latest | Hot
Please login before commenting.
Loading comments...
backend-refactor

Directory Structure

SKILL.md

Report

Notice
