Skip to content

refactor(product-domains): move pure product parsing policies#740

Merged
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/refactor-p9-product-domains-runtime
May 15, 2026
Merged

refactor(product-domains): move pure product parsing policies#740
limityan merged 1 commit into
GCWing:mainfrom
limityan:yanzhn/refactor-p9-product-domains-runtime

Conversation

@limityan
Copy link
Copy Markdown
Collaborator

Summary

  • Move MiniApp command and host allowlist policy helpers into bitfun-product-domains while keeping host dispatch execution in core.
  • Move Startchat and Git function-agent AI response value parsing into bitfun-product-domains while keeping AI calls, JSON extraction, and error mapping in core.
  • Add contract tests plus boundary-check and architecture-plan anchors, including the latest MiniApp customization metadata / permission-diff boundary.

Scope and Safety

  • This is a structure-only owner-boundary refactor; no product behavior, UI, command, or runtime execution semantics are intended to change.
  • bitfun-core still owns PathManager, host dispatch execution, worker / storage IO, customization draft storage and apply flow, Git / AI service adapters, prompt templates, and tool runtime.
  • The docs now explicitly mark runtime owner migration as deferred so this PR cannot be mistaken for a completed MiniApp/function-agent runtime migration.

Validation

  • node scripts/check-core-boundaries.mjs
  • rustfmt --edition 2021 --check on the touched Rust files
  • cargo test -p bitfun-product-domains --features product-full -- --nocapture
  • cargo test -p bitfun-product-domains --no-default-features
  • cargo check -p bitfun-core --features product-full
  • cargo test -p bitfun-core miniapp -- --nocapture
  • cargo test -p bitfun-core function_agent -- --nocapture
  • git diff --check

Move MiniApp allowlist and function-agent AI response parsing policy into bitfun-product-domains while keeping host execution, AI calls, JSON extraction, and runtime adapters in core.

Add contract tests, boundary-check anchors, and plan docs that mark runtime owner migration as still deferred.
@limityan limityan merged commit d58eb95 into GCWing:main May 15, 2026
4 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