Repository navigation
CLAUDE.md: what 0.8 changed, stated as it is now - #53
Merged
Merged
Conversation
A pass over the file against the code as it stands: - The preview is on Ctrl+F8 and shows PDF pages and video. - Three CMake targets, not two: the app is a static library that the program and the window test link. The tree gains child.cpp and menu.cpp; compare.cpp is no longer 'later'. - Rule 1 said sorting by size or date runs a background stat pass. It has not since the entries carry size and time (and now mode and owner) from the listing's own stat. - Rule 8: thumbnails exist. Rule 12: Name keeps a floor and the list scrolls sideways past it. - The budget table had the numbers of milestone 2. It has those of 0.8.2: more per entry (mode and owner), less after trimming. - Errors are returned as messages through completions; std::expected was the plan and is used nowhere. - The pre-release list names the window test. Still 350 lines. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KxLwQumbAZhrZpWaVaUo64
Milestones said 'since then: issues' and nothing about where the work is now: releases come as 0.x.y, and the next step is the AUR package. Non-goals ended on a sentence about how previews are made, which is a how and not a non-goal, and is rule 8. The list now names what was decided along the way as well: no dialogs (rule 10), no decoder linked into the process (rule 8), no sound or autoplay for video. Still 350 lines. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KxLwQumbAZhrZpWaVaUo64
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
A pass over CLAUDE.md against the code as it stands after 0.8.2. Docs only.
child.cppandmenu.cpp.std::expectedwas the plan and is used nowhere in the code; the convention now says what the code does. Say if you would rather keep it as the goal.Still 350 lines.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KxLwQumbAZhrZpWaVaUo64