Skip to content

fix: move agent-os-kernel to optional dep, correct version constraint - #120

Closed
imran-siddique wants to merge 1 commit into
mainfrom
fix/agent-os-kernel-dep
Closed

fix: move agent-os-kernel to optional dep, correct version constraint#120
imran-siddique wants to merge 1 commit into
mainfrom
fix/agent-os-kernel-dep

Conversation

@imran-siddique

Copy link
Copy Markdown
Member

Root cause

\�gent-os-kernel>=4.0\ was declared as a hard dependency in \pyproject.toml, but:

  1. Version 4.0 does not exist — latest is 3.7.0
  2. \scanner.py\ imports it with \ ry/except ImportError\ and a full no-op fallback — it was never a hard requirement

This caused \pip install -e '.[dev]'\ to fail on every CI matrix combination, blocking all 6 jobs.

Fix

  • Remove \�gent-os-kernel\ from required \dependencies\
  • Add new [agt]\ optional extra for users who want AGT catalog scanning: \�gent-os-kernel>=3.0\
  • Add \�gent-os-kernel>=3.0\ to [dev]\ so CI exercises the scanner code path

Test plan

  • \pip install -e '.[dev]'\ succeeds on Python 3.11/3.12/3.13
  • All 6 CI matrix jobs pass
  • \pip install -e '.[agt]'\ works for users who want the AGT scanner

🤖 Generated with Claude Code

agent-os-kernel is imported with try/except ImportError in scanner.py
and has a graceful no-op fallback — it was never a hard requirement.
The version constraint >=4.0 is also wrong: 4.0 does not exist (latest
is 3.7.0), causing pip install to fail on all CI matrix combinations.

Moves it to [project.optional-dependencies] as:
- �gt extra: for users who want AGT catalog scanning
- dev: so CI exercises the scanner path in tests

Fixes all 6 failing CI jobs from PR #114.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@imran-siddique

Copy link
Copy Markdown
Member Author

Superseded by #121. Making agent-os-kernel optional is not viable — proxy.py has hard top-level imports from agent_os.* so the gateway fails to start without it. #121 uses agent-os-kernel>=3.7 as a hard dep (the correct version constraint) and also fixes the AGT 3.7 API incompatibilities in tests and schema.

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