Skip to content

Default to disabled state, add library-level bulk toggles, and make Sync/Share header copy tab-aware#25

Merged
AbandonedCart merged 12 commits intomainfrom
copilot/set-default-share-off
Apr 23, 2026
Merged

Default to disabled state, add library-level bulk toggles, and make Sync/Share header copy tab-aware#25
AbandonedCart merged 12 commits intomainfrom
copilot/set-default-share-off

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 23, 2026

  • Resolve main.ts conflict block 1: merge updateTopDescription into setActiveTab; drop isShareTabContext
  • Resolve main.ts conflict block 2: keep setActiveTab(activeTab) in mountPage; remove orphaned listener setup
  • Add independent share state variables (shareDisabledSeriesIds, shareDisabledSeasonIds, etc.)
  • Add tab-aware state accessors; update all card and library-toggle functions to use them
  • Sync defaults to all-enabled (from config); Share always defaults to all-disabled on page mount (once per init(), not per search)
  • Update share() to use share-specific state sets
  • Reset share state in mountPage on navigation
  • ShareController.cs: remove redundant field; use primary constructor param directly
  • ShareSubmitRequest.cs: IReadOnlyCollection + [JsonInclude] + init; for true immutability
  • ShareSubmitResponse.cs: use init; instead of set; with corrected doc comments
  • SegmentStore.cs: add Id INTEGER PRIMARY KEY to SharedUploads + migration for existing DBs
  • ShareSubmissionService.cs: fix full-library scan (AncestorIds); fix partial-success race condition (record fingerprints immediately per-batch); ok flag starts false and is set true only when a batch actually succeeds
  • Build passes clean (0 errors, 0 warnings); tests pass
  • Parallel validation passed (0 CodeQL alerts)

Copilot AI and others added 8 commits April 23, 2026 01:29
Agent-Logs-Url: https://github.com/intro-skipper/skipme.db-plugin/sessions/9d0779b7-6463-4e22-bb0f-4020007a5609

Co-authored-by: AbandonedCart <1173913+AbandonedCart@users.noreply.github.com>
Agent-Logs-Url: https://github.com/intro-skipper/skipme.db-plugin/sessions/9d0779b7-6463-4e22-bb0f-4020007a5609

Co-authored-by: AbandonedCart <1173913+AbandonedCart@users.noreply.github.com>
Agent-Logs-Url: https://github.com/intro-skipper/skipme.db-plugin/sessions/9d0779b7-6463-4e22-bb0f-4020007a5609

Co-authored-by: AbandonedCart <1173913+AbandonedCart@users.noreply.github.com>
Agent-Logs-Url: https://github.com/intro-skipper/skipme.db-plugin/sessions/9d0779b7-6463-4e22-bb0f-4020007a5609

Co-authored-by: AbandonedCart <1173913+AbandonedCart@users.noreply.github.com>
Agent-Logs-Url: https://github.com/intro-skipper/skipme.db-plugin/sessions/9d0779b7-6463-4e22-bb0f-4020007a5609

Co-authored-by: AbandonedCart <1173913+AbandonedCart@users.noreply.github.com>
@AbandonedCart AbandonedCart marked this pull request as ready for review April 23, 2026 01:41
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented Apr 23, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (8 files)
  • SkipMe.Db.Plugin/Configuration/skipme-index.css
  • SkipMe.Db.Plugin/Configuration/skipme-index.js
  • SkipMe.Db.Plugin/Controllers/ShareController.cs
  • SkipMe.Db.Plugin/Models/ShareSubmitRequest.cs
  • SkipMe.Db.Plugin/Models/ShareSubmitResponse.cs
  • SkipMe.Db.Plugin/Services/ShareSubmissionService.cs
  • web/src/main.ts
  • web/src/styles/main.css

Reviewed by seed-2-0-pro-260328 · 410,227 tokens

Copilot AI and others added 2 commits April 23, 2026 01:58
@AbandonedCart
Copy link
Copy Markdown
Contributor

@copilot Resolve all the conflicts you created

@AbandonedCart AbandonedCart merged commit 0941afe into main Apr 23, 2026
1 check passed
@AbandonedCart AbandonedCart deleted the copilot/set-default-share-off branch April 23, 2026 02:48
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.

2 participants