Skip to content

Fix: stale references to renamed/moved code#39

Merged
selvamHexo merged 1 commit into
hexo-ai:mainfrom
benmaster82:fix/stale-docs-references
Jun 16, 2026
Merged

Fix: stale references to renamed/moved code#39
selvamHexo merged 1 commit into
hexo-ai:mainfrom
benmaster82:fix/stale-docs-references

Conversation

@benmaster82

@benmaster82 benmaster82 commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

After the modular refactor (#18) and the web visualizer (#19) several references in the docs and one user-facing error message went stale:

  • docs/configuration.md: env var defaults referenced SIA_META_PROFILE / SIA_TARGET_PROFILE; the real names are SIA_META_AGENT_PROFILE / SIA_TARGET_AGENT_PROFILE (see sia/config.py).
  • SECURITY.md: bypassPermissions section pointed at util.py, which is now a backward-compat shim; the option is in sia/agent_impls/claude.py.
  • sia/web/server.py and the README/configuration docs suggested installing a 'sia-agent[web]' extra that does not exist in pyproject.toml - fastapi and uvicorn are already in the base dependencies, so the dashboard is always available.
  • docs/architecture.md: directory tree still listed util.py as "Agent runner utilities" and pointed at META_AGENT_PROMPT / FEEDBACK_AGENT_PROMPT constants in orchestrator.py; the prompts now live in sia/prompts.py as build_meta_prompt / build_feedback_prompt, and the agent runners are under sia/agent_impls/.

No behavior change. ruff check and ruff format --check both pass.

After the modular refactor (hexo-ai#18) and the web visualizer (hexo-ai#19) several
references in the docs and one user-facing error message went stale:

- docs/configuration.md: env var defaults referenced SIA_META_PROFILE /
  SIA_TARGET_PROFILE; the real names are SIA_META_AGENT_PROFILE /
  SIA_TARGET_AGENT_PROFILE (see sia/config.py).
- SECURITY.md: bypassPermissions section pointed at util.py, which is now
  a backward-compat shim; the option is in sia/agent_impls/claude.py.
- sia/web/server.py and the README/configuration docs suggested
  installing a 'sia-agent[web]' extra that does not exist in
  pyproject.toml — fastapi and uvicorn are already in the base
  dependencies, so the dashboard is always available.
- docs/architecture.md: directory tree still listed util.py as "Agent
  runner utilities" and pointed at META_AGENT_PROMPT /
  FEEDBACK_AGENT_PROMPT constants in orchestrator.py; the prompts now
  live in sia/prompts.py as build_meta_prompt / build_feedback_prompt,
  and the agent runners are under sia/agent_impls/.

No behavior change. ruff check and ruff format --check both pass.
@benmaster82 benmaster82 changed the title fix stale references to renamed/moved code Fix: stale references to renamed/moved code Jun 15, 2026
@selvamHexo
selvamHexo self-requested a review June 16, 2026 15:47

@selvamHexo selvamHexo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@selvamHexo
selvamHexo merged commit 6ad5ad7 into hexo-ai:main Jun 16, 2026
9 checks passed
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.

2 participants