Skip to content

refactor(product-domains): externalize guarded domain helpers#713

Merged
limityan merged 2 commits into
GCWing:mainfrom
limityan:yanzhn/refactor-product-domains-guardrails
May 14, 2026
Merged

refactor(product-domains): externalize guarded domain helpers#713
limityan merged 2 commits into
GCWing:mainfrom
limityan:yanzhn/refactor-product-domains-guardrails

Conversation

@limityan
Copy link
Copy Markdown
Collaborator

Summary

  • Move MiniApp pure compiler/export/runtime/worker/storage/lifecycle/host-routing helpers and runtime/storage port contracts into bitfun-product-domains.
  • Move function-agent pure Git/Startchat helpers, project context analyzer, and Git/AI port contracts into bitfun-product-domains.
  • Keep bitfun-core as the runtime owner and compatibility facade, with core-owned MiniApp/function-agent adapter equivalence tests and updated decomposition guardrails.

Behavior and Boundary Guardrails

  • No product runtime owner migration in this PR.
  • PathManager, Git/AI service calls, prompt templates, built-in MiniApp seeding, host dispatch execution, process execution, worker pool/storage IO, ToolUseContext, tool manifest/exposure, and tool runtime remain core-owned.
  • Existing core import paths continue through wrappers/re-exports.

Validation

  • cargo test --workspace
  • node scripts/check-core-boundaries.mjs
  • git diff --check HEAD
  • git show --check --stat --oneline HEAD --

Note: workspace tests still report the existing CLI unused import warning at src/apps/cli/src/ui/theme.rs:4; this PR does not touch CLI.

Move MiniApp and function-agent pure contracts/helpers into bitfun-product-domains while keeping core as the runtime owner and compatibility facade.

Add core-owned adapter equivalence tests and update decomposition guardrails for the remaining runtime migration boundaries.
@limityan limityan force-pushed the yanzhn/refactor-product-domains-guardrails branch from 9e44f4a to dcfbb4c Compare May 14, 2026 07:03
@limityan limityan merged commit 2613f8b into GCWing:main May 14, 2026
8 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.

1 participant