Skip to content

refactor: unify host-native memory agents and npm package - #45

Merged
dongjiang1989 merged 7 commits into
mainfrom
fix/host-native-model-ports
Jul 28, 2026
Merged

refactor: unify host-native memory agents and npm package#45
dongjiang1989 merged 7 commits into
mainfrom
fix/host-native-model-ports

Conversation

@OLDyade

@OLDyade OLDyade commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • unify Extraction, Dream, and Skill Evolution on Pi Agent Core with host-native model and credential reuse across Pi, Hermes, OpenCode, and OpenClaw
  • consolidate the former adapters and Hermes distributions into the single public @iflytekopensource/memflywheel package
  • keep OpenCode final text responsive by submitting memory work from session.idle, and fix Hermes guard installation for current CLI permission semantics
  • update Quick Start, release workflows, Kind E2E images, examples, and package metadata for the single-package contract

Validation

  • pnpm run ci
  • npm latest dry-run
  • npm next dry-run
  • GitHub Packages dry-run with scope rewrite
  • clean tarball install/import and content scan
  • real local Pi, Hermes, OpenCode, and OpenClaw write + fresh-session recall
  • real OpenClaw Dream and Skill Evolution threshold runs

Release

  • version: 0.1.1
  • tag only after review and merge; this PR does not publish

OLDyade added 2 commits July 28, 2026 16:03
@OLDyade
refactor: unify host-native memory agents
06bc9e1 
Signed-off-by: MemScribe Maintainers <OLDyade@users.noreply.github.com>
@OLDyade
refactor: publish a single MemFlywheel package
17ce1b2 
Signed-off-by: MemScribe Maintainers <OLDyade@users.noreply.github.com>
github-code-quality[bot]
github-code-quality Bot found potential problems Jul 28, 2026
View reviewed changes
Comment thread packages/memflywheel/guard/__init__.py
try:
relative = root.relative_to(home).as_posix()
references.update({f"~/{relative}", f"$HOME/{relative}", f"${{HOME}}/{relative}"})
except ValueError:
OLDyade added 2 commits July 28, 2026 16:40
@OLDyade
test: isolate the Hermes installer contract
3fdc7f1 
Signed-off-by: MemScribe Maintainers <OLDyade@users.noreply.github.com>
@OLDyade
ci: use the current examples smoke entrypoint
99d73cb 
Signed-off-by: MemScribe Maintainers <OLDyade@users.noreply.github.com>
dongjiang1989
dongjiang1989 requested changes Jul 28, 2026
View reviewed changes

@dongjiang1989 dongjiang1989 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Need merge main branch

Comment thread CHANGELOG.md

### Added

- Kubernetes-level E2E CI workflow using kind and agent-sandbox CRDs. Deploys

@dongjiang1989 dongjiang1989 Jul 28, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@OLDyade Miss this chang docs

@OLDyade OLDyade Jul 28, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Restored the missing changelog entries with the current scope: the kind + agent-sandbox E2E covers Pi, Hermes, and OpenClaw, and the docs cover the 200-line direct-index limit and optional embedding setup. Commit: 29f2d37. The branch is already based on the latest main. Please re-review.

@dongjiang1989 dongjiang1989 Jul 28, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please double-check. At minimum, there is missing information in changelog.md. @OLDyade

@OLDyade OLDyade Jul 28, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Double-checked and fixed. I merged the latest main (including the DeepWiki badges) and restored the full changelog details: separate namespaces, custom images, full lifecycle validation, and offline/no-key execution. I only updated the host scope to include OpenClaw and the package wording to match the unified package. Commits: 0c05657, 5f6fc94. Please re-review.

@OLDyade
docs: restore unreleased changelog entries
29f2d37 
Signed-off-by: MemScribe Maintainers <OLDyade@users.noreply.github.com>
dongjiang1989
dongjiang1989 reviewed Jul 28, 2026
View reviewed changes
Comment thread scripts/publish-npm.mjs
const SCOPE_FROM = "@iflytekopensource/";
const SCOPE_TO = "@iflytek/";
const TARGET_PACKAGES = ["@iflytekopensource/adapters", "@iflytekopensource/hermes"];
const TARGET_PACKAGES = ["@iflytekopensource/memflywheel"];

@dongjiang1989 dongjiang1989 Jul 28, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is dry-run pass？

@OLDyade OLDyade Jul 28, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes — I ran the npm latest, npm next, and GitHub Packages dry-runs. All three passed, and nothing was published.

@OLDyade
OLDyade requested a review from dongjiang1989 July 28, 2026 09:20
OLDyade added 2 commits July 28, 2026 17:29
@OLDyade
Merge remote-tracking branch 'upstream/main' into fix/host-native-mod…
0c05657 
…el-ports

Signed-off-by: MemScribe Maintainers <OLDyade@users.noreply.github.com>
@OLDyade
docs: preserve the full E2E changelog details
5f6fc94 
Signed-off-by: MemScribe Maintainers <OLDyade@users.noreply.github.com>
FenjuFu
FenjuFu approved these changes Jul 28, 2026
View reviewed changes

@FenjuFu FenjuFu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the latest head (5f6fc94) after merging main. The previous merge-main blocker is resolved, and I found no new blocking issues in the package consolidation, host-native model bindings, extraction/locking changes, or associated tests and release updates. Approving, contingent on all required CI checks completing successfully.

dongjiang1989
dongjiang1989 approved these changes Jul 28, 2026
View reviewed changes

@dongjiang1989 dongjiang1989 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@dongjiang1989
dongjiang1989 merged commit 470d1fb into main Jul 28, 2026
10 checks passed
@dongjiang1989
dongjiang1989 deleted the fix/host-native-model-ports branch July 28, 2026 10:20
@dongjiang1989 dongjiang1989 mentioned this pull request Jul 29, 2026
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dongjiang1989 dongjiang1989 dongjiang1989 approved these changes
@FenjuFu FenjuFu FenjuFu approved these changes
+1 more reviewer
@github-code-quality github-code-quality[bot] github-code-quality[bot] left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@OLDyade @dongjiang1989 @FenjuFu