Repository navigation
refactor(src): group the flat src/ into area subfolders (move-only) - #487
Merged
Merged
Conversation
) git mv only, no content changes. Include paths and consumers follow in the next commits. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TdjopXZfsUiesYxYLYHEdH
…#486) build.bat gains WIND_INC (every src folder on the include path) and CORE_SRC (Wind.exe's source folders), and every target's explicit source list follows the new paths. Includes that reached a header through a path (../config.h, ../src/config.h) now name it bare. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TdjopXZfsUiesYxYLYHEdH
…le (#486) Code comments, docs, CLAUDE.md, tools, ui comments and the NSIS headers now cite src/<folder>/<file>. Chapter 01's source map gains a folder column, chapter 11 and CLAUDE.md state the bare-name include rule and WIND_INC, and the frozen specs and history READMEs note that their paths predate the regrouping. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01TdjopXZfsUiesYxYLYHEdH
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.
Closes #486
Move-only regrouping of the 118 files that sat directly in
src/, following chapter 01's source map. No code changes besides include lines and path-citing comments.Folder map
core/main.cpp,uiaccess_gate.h,uiaccess_token.h,idle_policy.h,sched_priority.h,tick_stats.h,logging.*,log_queue.h,test_telemetry.h,hitch_record.*,tick_span.h,com_util.h,reload_gate.hengine/magnifier_model.h,engine_pick.h,shell_desktop.h,launch_quiesce.hrender/render_engine.*,render_model.*,render_params.cpp,render_shaders.h,hlsl_compile.h,upscale_pass.*,upscale_shaders.h,upscale_nis.h,hdr_info.*,hdr_scale.h,band_window.h,png_dump.*transform/transform_model.*,transform.*,mag_host.*,tx_warm.h,comp_pin.*,mpo.h,mpo_boot.h,mpo_boot_logic.h,mpo_guard.h,native_cursor.h,zoom_ladder.h,dwm_watch.*color/color_filter.*,color_matrix.h,cursor_tint.*,cursor_tint_pixels.hinput/input_router.*,keybind_rules.h,pointer_binds.h,keyboard_pan.h,typing_key.h,event_order.h,swallow_ledger.hcursor/cursor_mapper.*,lock_detector.*,drag_follow.h,gain_learner.h,cursor_sprite.*,cursor_blanker.*,cursor_decode.*,sprite_layer.h,crosshair.*,cursor_lock.*,inspect_focus.h,game_cursor.h,pan_glide.htrack/focus_track.*,view_target.h,view_glide.h,detached_view.h,edge_pan.h,caret_rect.h,track_filter.h,java_bridge*,focus_identity.hzoom/zoom_controller.*config/config.*,config_path.h,ini_edit.*,profiles.*,profiles_io.h,profile_switch.htray/tray_host.*,tray_ipc.h,tray_items.*,tray_status.hplatform/installer_state.h,webview2_probe.hsrc/version.h,src/resource.h,src/wind.rc(version readers:tools/version.ps1,installer/wind.nsi,ui/vite.config.js, CI),src/tray_app/,src/config_ui/Include rule
A Wind header is always included by its bare file name (
#include "config.h"), fromsrc/andtests/alike.build.batdefinesWIND_INC(every src folder, plussrcitself) and passes it to everyclcall;CORE_SRClists Wind.exe's source folders once (app, uiaccess and check). Former path includes ("../config.h"in tray_app/config_ui,"../src/x.h"in tests) became bare. Exception: rc.exe gets no include path, sowind_tray.rckeeps"../resource.h"/"../version.h"(unchanged). Documented in CLAUDE.md and chapter 11 ("Source folders and includes"); chapter 01's source map gains a folder column.Commits
git mvonly (118 renames, 100% similarity).WIND_INC,CORE_SRC, explicit:test/tray/config source lists) and include lines.src/<file>; frozendocs/specsanddocs/historykeep old paths, their READMEs say so.Gates (all run on the final tree, clean
build\)build.bat check: exit 0build.bat test: exit 0, 643/643 test casesbuild.bat,build.bat uiaccess,build.bat tray,build.bat config(afternpm ci): exit 0npm run test:rules: 5/5;npx playwright test: 205 passedbuild.bat installer(withbuild\WindUA.exestaged from the uiaccess build as release.ps1 does) +tools\installer_check.ps1: all checks passed (install smoke skipped, needs elevation). tools\version.ps1; Get-WindVersion: 0.29.3src/<file>/src\<file>path outside frozen docs: 0 hitsgit diff -M --summary origin/main...HEAD: 118 renames; 97 at 100%, the other 21 are 95-99% because an include line or a comment citing asrc/path changedNot verified: deploy of the signed UIAccess build and a live run (no runtime change expected; binaries built from the same sources). CI's release/alpha workflows run build.bat unchanged.
🤖 Generated with Claude Code
https://claude.ai/code/session_01TdjopXZfsUiesYxYLYHEdH