Skip to content

feat(ppt): 支持个人 PPT 模板上传、管理与复用 - #386

Open
SuperstructureJH wants to merge 6 commits into
dataelement:v0.9.0from
SuperstructureJH:codex/ppt-personal-templates-pr
Open

feat(ppt): 支持个人 PPT 模板上传、管理与复用#386
SuperstructureJH wants to merge 6 commits into
dataelement:v0.9.0from
SuperstructureJH:codex/ppt-personal-templates-pr

Conversation

@SuperstructureJH

@SuperstructureJH SuperstructureJH commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

支持从 PPTX 导入、预览、保存和复用个人模板,保留来源素材、文本布局与可编辑元素,支持重命名、描述编辑和删除。

基于 v0.9.0@032dd37,使用 Harness 0.1.5-rc.2。保留当前版本的启动与会话协议,补齐 Connection 的 WebServer 依赖。独立工作区完成 789 项测试(全部通过)、类型检查和构建;真实 Desktop Host 鉴权及已安装功能路由回归通过。现有 Windows PR CI 已加入 v0.9.0 触发范围,相关 26 项回归通过。 PPT 运行包从本 PR 维护源码重建,16 套模板检查和包完整性校验通过。

真实模型端到端任务和原生 Office 编辑保存重开仍需单独验收。

yaojin3616 and others added 3 commits September 10, 2026 17:34
…ket (dataelement#351)

Uninstalling the plugin market ran `dsh plugin remove --workspace-root
dshmarket`, which edits only `dependencies` and `node_modules`. When
dshmarket is installed as a generation, `desired.json` is what the next
launch projects from — so prepareGenerationsForLaunch() rebuilt the
market during the restart the uninstall itself triggers, and the whole
operation looked like it had done nothing.

Branch on the install form instead: a generation goes through
uninstallGenerationPlugin() (disableGeneration + reproject), everything
else keeps the existing pnpm removal. The form is read BEFORE the
removal, so a generation whose disable fails reports a failure rather
than a success with the pointer still in place.

Not routed through removeProfilePluginCompletely(): dshmarket is a
CORE_BUNDLES name and beginRemoval() refuses it by design, which is the
guard that stops recovery and Safe Mode from tearing out a core bundle.

Fixes dataelement#330
…rom (dataelement#352)

* fix(market): install plugins from the registry the market read them from

A generation install had no way to learn a registry. The boundary keeps
only the package spec out of its arguments, its environment is the
service's closure rather than anything the caller passes, and the staging
`.npmrc` was written with no registry line — so pnpm fell back to the
user's `~/.npmrc`.

The market meanwhile picks WHICH version to install by reading metadata
from its region's registry (Tencent's mirror in the china region). Those
two registries diverge for minutes to hours after any publish, and while
they do, every update fails with ERR_PNPM_NO_MATCHING_VERSION or silently
resolves `@latest` to the older build and reports a stale update.

Resolve the market's own registry and pin the staging workspace to it, so
the half that chooses a version and the half that fetches it agree. A
project `.npmrc` outranks `~/.npmrc`, and unlike `npm_config_*` it does
not depend on a pnpm major still reading that env prefix.

The value is inferred from the market's persisted region because dshmarket
ships from npm on its own schedule and the fix cannot wait for a release
that passes it in — but `--registry=` is now read from the boundary's
arguments and wins, so a later market can state it outright. An already
named `npm_config_registry`, and a region on the default registry, are
both left alone: pinning npmjs.org over a private registry would break
installs that work today.

Also report pnpm's error CODE instead of the line after it. The trailing
"This error happened while installing a direct dependency of <path>" also
matches /error/i and, being last, displaced ERR_PNPM_NO_MATCHING_VERSION
from the log — which is why this could only be diagnosed by reproducing
the install by hand.

Fixes dataelement#337

* fix: align plugin updates with published market behavior

* test: make build approval fixture independent of Node shims

* test: canonicalize Windows temporary workspace paths
@Bian-Sh

Bian-Sh commented Sep 11, 2026 via email

Copy link
Copy Markdown

Adapt the feature to Harness 0.1.5-rc.2 and validate its independent Desktop composition.
@SuperstructureJH
SuperstructureJH changed the base branch from main to v0.9.0 September 11, 2026 09:25
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