Skip to content

revert: remove bundled DataLink semantic service (#81)#85

Closed
wangjk9527 wants to merge 5 commits into
datagallery-lab:mainfrom
wangjk9527:revert/3ffe7b7-datalink
Closed

revert: remove bundled DataLink semantic service (#81)#85
wangjk9527 wants to merge 5 commits into
datagallery-lab:mainfrom
wangjk9527:revert/3ffe7b7-datalink

Conversation

@wangjk9527

Copy link
Copy Markdown
Collaborator

Summary

  • Reverts 3ffe7b74518cb0bdf7c6ae6d565af6f789219b03 (feat: integrate DataLink semantic service), originally merged via #81.
  • Removes the in-tree services/datalink Python service, builtin DataLink server wiring, DataLink CI job, guides, and related npm scripts.
  • Conflict resolution note: later commits on main (especially native one-click deploy #82 / 31cc98f, and auth foundation) touch shared files (.env.example, CI, package.json, README/quick-start, scripts/stack-runner.mjs). To keep deploy/npm run start working, this revert keeps the stack runner / datalink-stack-config helpers used by deploy, but DATALINK_ENABLED=true now fails fast because the bundled service is gone. Deploy still defaults DataLink off.

Why

User-requested rollback of the DataLink integration on datagallery-lab/datafoundry via revert PR (no force-push to main).

Test plan

  • node --test scripts/datalink-stack-config.test.mjs scripts/stack-runtime-config.test.mjs scripts/deploy/*.test.mjs (93 passed)
  • CI green on this PR
  • Confirm DATALINK_ENABLED left unset/false still starts Web/API via npm run start / ./deploy.sh
  • Confirm enabling DataLink in deploy is unsupported after this change (expected fail-fast / missing service)

iancaoo and others added 5 commits July 21, 2026 11:19
提供 deploy.sh 原生部署链路(configure/install/build/start/stop/doctor/tui),
含端口选择、托管进程状态、密钥脱敏与健康检查;并按评审收紧端口菜单重试、
verifyPorts managedPorts 透传,以及 runtime log 的 0o600 创建权限。
实现 password 模式会话/CSRF/注册策略,并修复 HITL 事件桥、cookie Path
与缺失注册策略静默 open 等确定性缺陷。脚本侧统一正式认证客户端,并为
FORMAL_HTTP_AUTH_TARGETS 增加 node --check 语法门禁。
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