umputun / revdiff

89%

master: 90%

LAST BUILD ON BRANCH fix/theme-persist-ini-section

branch: fix/theme-persist-ini-section

CHANGE BRANCH

x

Reset

Sync Branches

• fix/theme-persist-ini-section
• add-collapsed-flag
• all-files-mode
• annotation-list-popup
• annotation-rowcache
• ci-shellcheck-yaml
• code-smells-cleanup
• codex-plugin
• collapsed-diff-mode
• commit-info-popup
• community-themes
• compact-diff-mode
• compact-improve
• compare-flag
• custom-keybindings
• daulet/new_file
• daulet/word_highlight
• dependabot/go_modules/github.com/alecthomas/chroma/v2-2.24.1
• dependabot/go_modules/github.com/alecthomas/chroma/v2-2.25.0
• dependabot/go_modules/github.com/alecthomas/chroma/v2-2.26.1
• dependabot/go_modules/github.com/charmbracelet/x/ansi-0.11.7
• dependabot/go_modules/github.com/mattn/go-runewidth-0.0.23
• dependabot/go_modules/github.com/mattn/go-runewidth-0.0.24
• diff-search
• discard-quit
• docs/pi-package-review-fixes
• eager-commit-fetch
• exit-code-on-annotations
• external-editor-annotations
• feat/annotation-marker
• feat/cmux-support
• feat/history-skill-fallback
• feat/jj-support
• feat/launch-script-improvements
• feat/navigation-scrollbar
• feat/one-light-chroma-style
• feat/open-editor-keymap
• feat/opencode-config-docs
• feat/overlay-mouse-support
• feat/preload-annotations
• feat/review-info-overlay
• feat/reviewed-files-and-status-indicators
• feat/scroll-indicators
• feat/scrollbar-thumb
• feat/stdin-scratch-buffer
• feat/toggle-tree-pane
• feat/tree-focus-diff-scroll
• feat/untracked-flag
• feat/wheel-scroll-viewport-only
• feat/word-diff-highlighting
• feat/word-diff-opt-in
• feat/zed-revdiff-task
• feature/annotation-navigation
• feature/binary-file-detection
• feature/layout-agnostic-keys
• feature/layout-agnostic-keys-v2
• feature/new-file-and-untracked-support
• feature/range-hunk-annotations
• fix-ctrl-du-half-page
• fix-file-annotation-bugs
• fix-initial-loading-flash
• fix-marketplace-name-in-docs
• fix-prefix-fg-on-highlighted-lines
• fix-tmux-title-compat
• fix-tree-wheel-single-step
• fix-wheel-scroll-coalesce
• fix-wrap-right-padding
• fix/124-page-cursor-relative-position
• fix/center-hunk-in-viewport
• fix/cmux-close-surface-recycled-ref
• fix/cmux-ghostty-detection
• fix/cmux-send-stdout-leak
• fix/diff-viewport-visual-height-scroll
• fix/ghostty-improvements
• fix/issue-203-zellij-popup-size
• fix/kitty-ssh-remote-launch
• fix/launcher-argument-quoting
• fix/only-dot-prefix
• fix/overlay-path-inheritance
• fix/pi-revdiff-review-trigger
• fix/staged-only-empty-diff
• fix/subdirectory-support
• global-hunk-navigation
• hunk-keyword-expansion
• include-filter
• intraline-word-diff
• iterm2-osascript
• launcher-override-chain
• leader-chord-keybindings
• line-numbers
• markdown-toc-navigation
• master
• mercurial-support
• mouse-support
• move-code-to-app
• only-filter
• overlay-extraction
• pi-revdiff-claude-workflow
• refs/tags/v0.1.0
• refs/tags/v0.1.1
• refs/tags/v0.10.0
• refs/tags/v0.11.0
• refs/tags/v0.12.0
• refs/tags/v0.13.0
• refs/tags/v0.14.0
• refs/tags/v0.14.1
• refs/tags/v0.15.0
• refs/tags/v0.15.1
• refs/tags/v0.15.2
• refs/tags/v0.15.3
• refs/tags/v0.16.0
• refs/tags/v0.16.1
• refs/tags/v0.17.0
• refs/tags/v0.18.0
• refs/tags/v0.18.1
• refs/tags/v0.19.0
• refs/tags/v0.19.1
• refs/tags/v0.19.2
• refs/tags/v0.2.0
• refs/tags/v0.2.1
• refs/tags/v0.2.2
• refs/tags/v0.2.3
• refs/tags/v0.2.4
• refs/tags/v0.20.0
• refs/tags/v0.21.0
• refs/tags/v0.22.0
• refs/tags/v0.23.0
• refs/tags/v0.24.0
• refs/tags/v0.25.0
• refs/tags/v0.25.1
• refs/tags/v0.26.0
• refs/tags/v0.26.1
• refs/tags/v0.27.0
• refs/tags/v0.27.1
• refs/tags/v0.28.0
• refs/tags/v0.3.0
• refs/tags/v0.4.0
• refs/tags/v0.4.1
• refs/tags/v0.4.2
• refs/tags/v0.5.0
• refs/tags/v0.6.0
• refs/tags/v0.7.0
• refs/tags/v0.7.1
• refs/tags/v0.7.2
• refs/tags/v0.8.0
• refs/tags/v0.9.0
• refs/tags/v1.0.0
• refs/tags/v1.1.0
• refs/tags/v1.1.1
• refs/tags/v1.2.0
• refs/tags/v1.2.1
• refs/tags/v1.3.0
• refs/tags/v1.4.0
• refs/tags/v1.4.1
• refs/tags/v1.5.0
• refs/tags/v1.5.1
• refs/tags/v1.6.0
• refs/tags/v1.6.1
• reload
• rename-aware-diffs
• revert/197-opencode-symlink
• revert/opencode-setup-symlink
• review-findings
• review-history
• search-history
• shell-completions
• sidepane-extraction
• single-column-line-numbers-full-context
• single-file-mode
• skill-opencode-timeout-fix
• skill-scripts-hg-jj
• skill-use-claude-skill-dir
• status-bar-line-number
• status-line-help-overlay
• structural-refactor
• style-extraction
• theme-support
• two-ref-diff
• ui-package-split
• update-pi-extension-agent-tool
• vim-motion-preset
• widen-annot-list-popup
• word-wrap
• worddiff-extraction
• worktree-bugfix
• worktree-no-git-file-review
• wrap-marker-style

Build # 24880590308

Build Type

Pull #149

github

Committed by umputun

Commit Message

fix(themes): remove all misplaced theme lines during heal

Addresses Copilot review finding on #149. The previous heal logic found
the first `theme = ...` occurrence and stopped. When a user hand-repaired
a config corrupted by the pre-fix bug by adding a valid `theme = ...`
in [Application Options] without deleting the stray inside
[color options], the patch updated the valid line in place and left
the stray, so go-flags kept erroring with `unknown option: theme` on
next startup.

scanThemeLine -> scanThemeLines now walks the full file and reports
the first default-scope index separately from every stray. The patch
loop deletes strays in reverse order (to keep earlier indices stable)
before replacing/inserting the canonical entry.

Add TestPatchConfigTheme_removesStrayDuplicates regression test and
extend the testdata round-trip table with duplicate.ini, a fixture
shaped like the hand-repair scenario, to assert the file parses cleanly
via flags.NewIniParser after the heal.

Related to #148

Pull Request Pull Request #149: fix(themes): persist theme into Application Options section

Coverage Stats

48 of 52 new or added lines in 1 file covered. (92.31%)

8441 of 9437 relevant lines covered (89.45%)

551.27 hits per line