Skip to content

Add remote skill sync#2971

Open
YOMXXX wants to merge 1 commit into
multica-ai:mainfrom
YOMXXX:codex/p2-remote-skill-sync
Open

Add remote skill sync#2971
YOMXXX wants to merge 1 commit into
multica-ai:mainfrom
YOMXXX:codex/p2-remote-skill-sync

Conversation

@YOMXXX

@YOMXXX YOMXXX commented May 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a sync-from-origin endpoint for imported skills
  • replace skill content/files from the remote source and update origin metadata
  • add a skill detail sync action with dirty-state protection

Tests

  • cd server && go test ./internal/handler -run 'TestSkillOriginSourceURL|TestImportedSkillFilesFiltersUnsafePaths|TestSyncSkillFromOriginRejectsManualSkill' -count=1
  • cd server && go test ./cmd/server -count=1
  • pnpm --filter @multica/core typecheck
  • pnpm --filter @multica/views typecheck
  • jq empty packages/views/locales/en/skills.json packages/views/locales/zh-Hans/skills.json
  • git diff --check

@vercel

vercel Bot commented May 21, 2026

Copy link
Copy Markdown

@YOMXXX is attempting to deploy a commit to the IndexLabs Team on Vercel.

A member of the Team first needs to authorize it.

@YOMXXX YOMXXX mentioned this pull request May 21, 2026
6 tasks
@YOMXXX
YOMXXX force-pushed the codex/p2-remote-skill-sync branch from c0250d0 to eb1947d Compare May 31, 2026 09:18
@YOMXXX
YOMXXX force-pushed the codex/p2-remote-skill-sync branch from eb1947d to e2d3249 Compare July 19, 2026 12:22
@YOMXXX
YOMXXX force-pushed the codex/p2-remote-skill-sync branch from e2d3249 to 9c7fe79 Compare July 19, 2026 12:34
@YOMXXX

YOMXXX commented Jul 19, 2026

Copy link
Copy Markdown
Contributor Author

Rebased onto current upstream/main and force-pushed 9c7fe79d7; the PR is merge-clean again.

Conflict resolution kept the current skill label routes and added this PR sync route: POST /api/skills/{id}/sync in the same skill detail route group.

I also fixed the frontend CI locale parity failure by adding the missing Japanese strings for the new sync action.

Local verification:

  • cd server && go test ./internal/handler -run 'TestSyncSkillFromOriginRejectsManualSkill|TestImportSkill' -count=1
  • pnpm --filter @multica/core typecheck
  • pnpm --filter @multica/views typecheck
  • pnpm --filter @multica/views exec vitest run locales/parity.test.ts
  • jq empty packages/views/locales/en/skills.json packages/views/locales/ko/skills.json packages/views/locales/zh-Hans/skills.json packages/views/locales/ja/skills.json
  • git diff --check

GitHub CI is green across backend, frontend, mobile, changes, and installers. Ready for review.

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