feat(plugin-history-sync): expose URL resolver (FEP-2602) - #743
Conversation
🦋 Changeset detectedLatest commit: f017157 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
commit: |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
stackflow-docs | 18914c4 | Commit Preview URL | Aug 04 2026, 12:31 PM |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (5)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughThis PR adds ChangesHistory Sync URL Resolver
Estimated code review effort: 3 (Moderate) | ~25 minutes Sequence Diagram(s)sequenceDiagram
participant historySyncPlugin
participant HistorySyncUrlResolver
participant HistorySyncController
historySyncPlugin->>HistorySyncUrlResolver: create resolver from routes and location
historySyncPlugin->>HistorySyncUrlResolver: resolveEntryUrl(initialContext)
HistorySyncUrlResolver-->>historySyncPlugin: entry URL
historySyncPlugin->>HistorySyncController: provide makeActivityUrl
HistorySyncController-->>historySyncPlugin: activity path for push or replace
Possibly related PRs
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ENvironmentSet
left a comment
There was a problem hiding this comment.
이번 라운드는 문제 정의 / 해결 방향 / 공개 인터페이스 및 규약을 봤습니다.
문제 정의(“URL 해석 규칙의 사본이 실제 동작과 어긋난다”)와 해결 방향(규칙을 추출하지 않고, 플러그인이 실제로 쓰는 인스턴스를 그대로 노출해 내부 호출부 4곳을 전부 그것으로 교체)에는 동의합니다.
공개 인터페이스에서 한 가지 요청 사항을 인라인으로 남겼습니다. 반영 후 이 스레드에 답장 주세요.
Removed section on reusing URL semantics from README.
Uh oh!
There was an error while loading. Please reload this page.