mirror of
https://github.com/bytedance/deer-flow.git
synced 2026-07-27 08:28:00 +00:00
7aa314b4c1
* feat: add lark cli integration * fix: polish lark integration actions * feat: support lark incremental permissions * fix: detect lark authorization completion * fix: harden lark integration install * feat: expand lark auth scopes and reuse host auth in sandbox Default lark auth to least-privilege (recommend=false, base sign-in only) and expose the full set of lark-cli --domain business domains as native --domain grants instead of a 4-domain read-only mapping. Resolve the skill pack from the latest larksuite/cli GitHub release at install time with content-hash integrity, and surface version/runtime drift in status. Share the per-user lark-cli config/data profile between the Gateway Settings auth flow and agent conversations by mounting the integration dirs into the AIO sandbox and injecting the matching env for lark-cli commands, with an allowlisted extra_mounts path in the provisioner/K8s backend and traversal guards on integration paths. * style: fix lint issues from ruff and prettier Sort imports in the provisioner PVC test and re-wrap two long i18n description strings to satisfy backend ruff and frontend prettier CI. * fix(lark): address managed integration review feedback * fix(frontend): stabilize integrations settings e2e * test(sandbox): isolate remote backend legacy visibility check * test: fix backend unit failures after merge * Harden Lark integration review fixes * Format Lark integration E2E test * fix(lark): harden sandbox credential exposure and status disclosure Address willem_bd's security review on PR #3971: - Mount the per-user lark-cli config dir (long-lived appSecret) read-only into the AIO sandbox; only the refreshable-token data dir stays writable. - Redact host filesystem paths (install_path, cli.path) from GET /lark/status and the config/auth complete responses for non-admin callers, fail-closed on any auth error. - Document the npm postinstall trade-off (--ignore-scripts is not viable because @larksuite/cli fetches its platform binary in postinstall). - Document the sandbox credential trust boundary in AGENTS.md and README, pointing at the sidecar-broker follow-up (#4338). --------- Co-authored-by: Willem Jiang <willem.jiang@gmail.com>
45 lines
1.5 KiB
Python
45 lines
1.5 KiB
Python
"""Gateway import regression tests."""
|
|
|
|
from __future__ import annotations
|
|
|
|
import os
|
|
import subprocess
|
|
import sys
|
|
from pathlib import Path
|
|
|
|
|
|
def _gateway_import_env() -> dict[str, str]:
|
|
backend_root = Path(__file__).resolve().parents[1]
|
|
harness_root = backend_root / "packages" / "harness"
|
|
python_path_entries = [str(backend_root), str(harness_root)]
|
|
if existing_python_path := os.environ.get("PYTHONPATH"):
|
|
python_path_entries.append(existing_python_path)
|
|
return {**os.environ, "PYTHONPATH": os.pathsep.join(python_path_entries)}
|
|
|
|
|
|
def test_gateway_app_imports_first_without_subagent_import_cycle() -> None:
|
|
"""The replay gateway imports app.gateway.app in a clean process."""
|
|
result = subprocess.run(
|
|
[sys.executable, "-c", "from app.gateway.app import app"],
|
|
capture_output=True,
|
|
text=True,
|
|
env=_gateway_import_env(),
|
|
)
|
|
assert result.returncode == 0, result.stderr
|
|
|
|
|
|
def test_subagent_package_public_executor_exports_are_lazy_importable() -> None:
|
|
"""The package-level executor exports must not re-enter their own import."""
|
|
result = subprocess.run(
|
|
[
|
|
sys.executable,
|
|
"-c",
|
|
"from deerflow.subagents import SubagentExecutor, SubagentResult; print(SubagentExecutor.__name__, SubagentResult.__name__)",
|
|
],
|
|
capture_output=True,
|
|
text=True,
|
|
env=_gateway_import_env(),
|
|
)
|
|
assert result.returncode == 0, result.stderr
|
|
assert "SubagentExecutor SubagentResult" in result.stdout
|