Skip to content

migrate agent harness framework to burr#7

Merged
ethenotethan merged 2 commits intomainfrom
migrate-to-burr
Apr 18, 2026
Merged

migrate agent harness framework to burr#7
ethenotethan merged 2 commits intomainfrom
migrate-to-burr

Conversation

@ethenotethan
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread agent/agent.py Fixed
…list (#8)

* pin GitHub Actions to full commit SHAs for security

* add url parsing to avoid substring attack

* remove legacy libraries/applications split, unify to flat components list

The discovery engine wrote components.json with separate 'libraries' and
'applications' keys, but the Burr analysis pipeline read a 'components'
key that didn't exist — causing every component to be silently skipped
during analysis (0 components analyzed).

Instead of patching the reader, remove the entire legacy binary split:

- Discovery now writes flat {"components": [...]} format
- Remove Library, Application, BaseComponent, KnowledgeBasis classes
- Remove 'classification' field and _legacy_classification() from Component
- Remove libraries_used, internal_applications compat fields
- Rename ApplicationEdge to ComponentEdge
- Remove two-phase get_analysis_order() from DependencyGraph
- Replace libraries_count/applications_count with components_count in manifest
- Validator now checks all components in depth order, not just libraries
- Simplify all readers (burr_app.py, cli.py, scripts) to use 'components' key
- Rewrite utils/dependency_graph.py and scripts to use Component type
- Update all tests for new format
Copy link
Copy Markdown
Member

@0xClandestine 0xClandestine left a comment

Choose a reason for hiding this comment

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

lgtm

@ethenotethan ethenotethan merged commit 2ac342e into main Apr 18, 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.

3 participants