Skip to content

Make CQ propose local-first under team sync.#168

Open
jonathanKingston wants to merge 1 commit intomozilla-ai:mainfrom
jonathanKingston:jkt/dual-write-local-team-proposals
Open

Make CQ propose local-first under team sync.#168
jonathanKingston wants to merge 1 commit intomozilla-ai:mainfrom
jonathanKingston:jkt/dual-write-local-team-proposals

Conversation

@jonathanKingston
Copy link
Copy Markdown
Contributor

Keep durable local knowledge units even when team sync is enabled by writing locally before team submission and tracking sync state for retries instead of deleting synced local rows. Preserve KU IDs across local and team stores so merged reads and later propagation remain coherent.

Happy to open an issue up to discuss :).

Keep durable local knowledge units even when team sync is enabled by writing locally before team submission and tracking sync state for retries instead of deleting synced local rows. Preserve KU IDs across local and team stores so merged reads and later propagation remain coherent.

Made-with: Cursor
@jonathanKingston
Copy link
Copy Markdown
Contributor Author

@peteski22 do you want issues to discuss all of these prs? (I think this might be the most contentious/discussion worthy) :)

@peteski22 peteski22 self-assigned this Mar 31, 2026
@peteski22
Copy link
Copy Markdown
Collaborator

@peteski22 do you want issues to discuss all of these prs? (I think this might be the most contentious/discussion worthy) :)

Hi @jonathanKingston thanks for the comment, yes I think this is more a discussion via issue than a PR please.

My feedback would be that we intentionally made the decision to try and eliminate duplicates where possible, so draining to the remote would remove local KUs.

It's not to say we shouldn't potentially cache them in the future, but it seems the cleanest as we're trying to iron out the graduation process etc.

@peteski22 peteski22 added waiting-for-response PR or issue that is awaiting a response from the author. do-not-merge labels Apr 7, 2026
@jonathanKingston
Copy link
Copy Markdown
Contributor Author

Will follow up with an issue, thanks for taking a look. Keeping open until I do that :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge waiting-for-response PR or issue that is awaiting a response from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants