Skip to content

chore: black format drift + drop "framework" language in docs - #29

Merged
rdwj merged 2 commits into
mainfrom
chore/black-format-and-toolkit-reframe-readme
May 4, 2026
Merged

chore: black format drift + drop "framework" language in docs#29
rdwj merged 2 commits into
mainfrom
chore/black-format-and-toolkit-reframe-readme

Conversation

@rdwj

@rdwj rdwj commented May 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

Two unrelated cleanup commits caught by /session-close:

  1. style: Apply black formatting drift fix — four source files had drifted from the project's Black config (line length 100). CI's black --check step on test.yml was failing on main. Pure mechanical reformat; no logic changes.

    • src/fips_agents_cli/commands/create.py
    • src/fips_agents_cli/commands/vendor.py
    • src/fips_agents_cli/tools/git.py
    • src/fips_agents_cli/tools/project.py
  2. docs: Drop "framework" language in favor of fipsagents — aligns README.md and CLAUDE.md with the toolkit reframe shipped to docs: reframe fips-agents as a toolkit, not a framework examples#25 and chore(makefile): rename update-framework → update-fipsagents agent-template#150. Four lines in README ("vendored framework source", "framework code", "framework version", "framework source locally") rewritten to use "fipsagents" instead. CLAUDE.md additionally fixes two stale src/base_agent/ references that described a path which never existed in the agent-loop template — the template ships with src/agent.py directly.

Why

fips-agents is a scaffolding toolkit (CLI + composable templates), not a framework. Calling the CLI/ecosystem "the framework" sets the wrong expectation about runtime model. The pieces that are framework-shaped (BaseAgent, FastMCP) are inherited or template-internal — fips-agents-cli itself doesn't impose a runtime.

Test plan

  • black --check src tests passes (4 files reformatted, 25 unchanged)
  • ruff check src tests passes
  • pytest passes (207 passed, 0 failed)
  • gitleaks detect finds no secrets
  • No remaining "framework" hits in README.md or CLAUDE.md (other than xml-analysis-framework, an external project name, which is not present in this repo's docs)

rdwj added 2 commits May 4, 2026 09:18
Four source files had drifted from the project's Black configuration
(line length 100). Running `black src tests` produced these changes
mechanically; no logic changed.

Reformatted:
- src/fips_agents_cli/commands/create.py
- src/fips_agents_cli/commands/vendor.py
- src/fips_agents_cli/tools/git.py
- src/fips_agents_cli/tools/project.py

CI's `black --check` step on test.yml will now pass.

Assisted-by: Claude Code (Opus 4.7)
Aligns README.md and CLAUDE.md with the toolkit reframe shipped to
fips-agents/examples#25 and fips-agents/agent-template#150. fips-agents
is a scaffolding toolkit (CLI + composable templates), not a framework
— calling it that creates the wrong expectation about runtime model.

README changes:
- "vendored framework source" → "vendored fipsagents source"
- "fipsagents framework source"/"framework code" → "fipsagents
  source"/"fipsagents code"
- "framework version"/"framework source locally" → "fipsagents
  version"/"fipsagents source locally"

CLAUDE.md also fixes two stale `src/base_agent/` references that
described a path which never existed in the agent-loop template — the
template ships with `src/agent.py` directly, no project-named module
wrapper. Both lines also dropped "framework" wording in the same edit.

Assisted-by: Claude Code (Opus 4.7)
@rdwj
rdwj merged commit d5aa8d7 into main May 4, 2026
3 checks passed
@rdwj
rdwj deleted the chore/black-format-and-toolkit-reframe-readme branch May 4, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant