Skip to content

Add CLAUDE.md guidance for agentic development with Claude Code #1690

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
glwagner wants to merge 65 commits into from
Closed

Add CLAUDE.md guidance for agentic development with Claude Code #1690

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
65 commits
Select commit Hold shift + click to select a range
4622d6a
+*Fix 3-equation ice-ocean flux iteration (#972)
Hallberg-NOAA Oct 1, 2025
8d75416
Fixes shelfwave failure in debug mode
kshedstrom Oct 10, 2025
f264134
Make sure reversed segments get rotated.
kshedstrom Oct 24, 2025
75fa240
Bugfix for default TIDES_ANSWER_DATE in SAL
herrwang0 Sep 26, 2025
257650d
Fixes a typo in Recon1d PPM limiter
adcroft Oct 21, 2025
5994b9a
Frequency-dependent drag in tensor form
c2xu Oct 8, 2025
8e73194
Adds a PLM reconstruction scheme using least squares for the slope
adcroft Nov 4, 2025
e0f2469
Corrected unit descriptions in 64 comments
Hallberg-NOAA Nov 25, 2025
970e5af
*Update TC testing parameters for late 2025
Hallberg-NOAA Nov 25, 2025
96e186f
Added frazil to ice shelf (#985)
alex-huth Dec 2, 2025
196e491
Extend the PGF reconstruction to allow PLM-WLS
adcroft Nov 5, 2025
f2c8917
Spatially varying bottom drag coefficient (#983)
c2xu Dec 3, 2025
ba6032b
Correction on total column thickness for wetting
herrwang0 Sep 24, 2025
7d90219
Allow overshoot for for grounding test
adcroft Dec 4, 2025
ff7ec3e
Initialize and integer only set on root_PE()
adcroft Dec 4, 2025
367de74
Fix an uninitialized float in set_viscous_ML()
adcroft Dec 4, 2025
0f26d69
Add floor to "h_marg" in continuity_PPM
adcroft Dec 4, 2025
67f7637
*+Fix CHANNEL_DRAG with bathymetry above sea level
Hallberg-NOAA Dec 3, 2025
50b2c3b
Corrected 66 unit descriptions in comments
Hallberg-NOAA Dec 1, 2025
c008a01
Fix for ice-shelf friction velocity bugs (#995)
alex-huth Dec 9, 2025
9e48380
(+) Decouple FMS infra from framework
marshallward Dec 5, 2025
eac630c
Added latent heat flux from ice shelf to ocean fluxes
alex-huth Dec 10, 2025
760f765
Fixes wrong number of levels in z-coord diags
adcroft Dec 19, 2025
d4a5fdd
Fix bug in registration of ALE sponge diagnostics for generic tracers…
andrew-c-ross Dec 26, 2025
bf1716f
Fix reference of (rarely) unassociated pointer
adcroft Dec 22, 2025
7ee13cd
Adds 5 CMIP7 diagnostics for vertically integrated heat/salt content
adcroft Dec 22, 2025
3accd27
Optimized the ice-shelf CG scheme by reducing the number of times rep…
alex-huth Jun 9, 2025
9030b2e
Check that frazil is allocated before adding it to ice-shelf water fl…
alex-huth Dec 29, 2025
a742bc3
Added melt_mask for ice shelves
alex-huth Sep 25, 2025
cc3f1a6
Added melt_mask to ice-shelf restart
alex-huth Dec 4, 2025
b99fbd5
comments and units
alex-huth Jan 6, 2026
d5ecb10
subroutine ice_shelf_solve_inner: Completed variable descriptions and…
alex-huth Jan 7, 2026
f761134
+Add trim_trailing_commas and ints_to_string
Hallberg-NOAA Jan 2, 2026
9f7aeb9
Call trim_trailing_commas from register_diag_field
Hallberg-NOAA Jan 2, 2026
d3de0cb
Refactor nsten_halo in routine advect_tracer
herrwang0 Jan 5, 2026
a8f6945
Fix OBC indexing bug in MOM_tracer_advect
herrwang0 Jan 5, 2026
d19d917
MOM_interpolate: use get_axis_size()
marshallward Jan 6, 2026
b03a7fb
ice-ocean-nolib: Fix SIS2 paths
marshallward Jan 13, 2026
2076dc7
Correct the path to the Icepack interfaces
Hallberg-NOAA Jan 13, 2026
5875be7
Delete unneeded masks args from 25 post_data calls
Hallberg-NOAA Dec 29, 2025
7343006
Add 2D meanSL field
herrwang0 Sep 17, 2025
f1b9c45
Use meanSL to calcualte mean column thickness
herrwang0 Sep 18, 2025
cbcc012
Modify max_depth calculation using meanSL
herrwang0 Oct 2, 2025
c278715
+Fix how missing values are handled in post_data
Hallberg-NOAA Dec 29, 2025
2e8793f
Remove ice-sheet melting/freezing contribution to fluxes%latent becau…
alex-huth Jan 8, 2026
7d00286
Merge branch 'main' into dev/gfdl
marshallward Feb 1, 2026
8f92390
+Add G%IdxCu_OBCmask and G%IdyCv_OBCmask
Hallberg-NOAA Jan 17, 2026
99c32b7
Use G%IdxCu_OBCmask in 7 places
Hallberg-NOAA Jan 18, 2026
5801592
Add option to scale tidal amplitude for bottom ustar. (#1016)
breichl Feb 1, 2026
ac64aaa
Add vertical tracer flux diagnostic for dye tracers (#1022)
aakashsane Feb 2, 2026
ff6b3b0
Regroup MOM_initialize_fixed params in param_doc
herrwang0 Jan 16, 2026
27f4243
Minor open_boundary_config refactor
herrwang0 Jan 16, 2026
a5da88f
Refactor gradKE with simpler treatment of OBCs
Hallberg-NOAA Jan 19, 2026
85aacaa
Autoconf: Build dependencies as libraries
marshallward Dec 30, 2025
0e9061e
ac: Configure use of pkg/ source
marshallward Feb 4, 2026
ae752b6
Deleted about 170 unused internal variables
Hallberg-NOAA Feb 5, 2026
0f977a4
Decrease open_boundary_config indentation
herrwang0 Feb 7, 2026
412c177
Correct the unit of diagnostic "eta_cor"
herrwang0 Feb 5, 2026
aefec49
Merge [uv]bt_st and [uv]bt_first in btstep
herrwang0 Jan 31, 2025
cdc4709
Add CLAUDE.md development guide for MOM6
glwagner Feb 27, 2026
e6a0686
Add 'Purpose of this document' section to CLAUDE.md
glwagner Feb 27, 2026
4c83407
Update CLAUDE.md
glwagner Feb 27, 2026
7b089c3
Add GOTM link and key references pointer to CLAUDE.md
glwagner Feb 27, 2026
a4adbdf
Note CLAUDE.md as a living document; add Common Claude Mistakes section
glwagner Feb 27, 2026
46c918b
Expand CLAUDE.md with patterns from ~25 additional PR reviews
glwagner Feb 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
97 changes: 37 additions & 60 deletions .github/workflows/verify-linux.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- name: Check white space (non-blocking)
run: |
Expand Down Expand Up @@ -50,13 +48,14 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile FMS
run: make -C .testing -j build/deps/lib/libFMS.a
- name: Compile dependencies
run: |
make -C .testing -j build/deps/lib/libFMS.a
make -C .testing -j PKG= build/deps/lib/libgsw.a
make -C .testing -j PKG= build/deps/lib/libcvmix.a

- name: Compile MOM6 with symmetric indexing
run: make -C .testing -j build/symmetric/MOM6
Expand All @@ -75,13 +74,14 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile FMS
run: make -C .testing -j build/deps/lib/libFMS.a
- name: Compile dependencies
run: |
make -C .testing -j build/deps/lib/libFMS.a
make -C .testing -j PKG= build/deps/lib/libgsw.a
make -C .testing -j PKG= build/deps/lib/libcvmix.a

- name: Compile MOM6 with asymmetric indexing
run: make -C .testing -j build/asymmetric/MOM6
Expand All @@ -100,13 +100,14 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile FMS
run: make -C .testing -j build/deps/lib/libFMS.a
- name: Compile dependencies
run: |
make -C .testing -j build/deps/lib/libFMS.a
make -C .testing -j PKG= build/deps/lib/libgsw.a
make -C .testing -j PKG= build/deps/lib/libcvmix.a

- name: Compile MOM6 with bit-reproducible optimization
run: make -C .testing -j build/repro/MOM6
Expand All @@ -125,13 +126,14 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile FMS
run: make -C .testing -j build/deps/lib/libFMS.a
- name: Compile dependencies
run: |
make -C .testing -j build/deps/lib/libFMS.a
make -C .testing -j PKG= build/deps/lib/libgsw.a
make -C .testing -j PKG= build/deps/lib/libcvmix.a

- name: Compile MOM6 supporting OpenMP
run: make -C .testing -j build/openmp/MOM6
Expand All @@ -151,12 +153,10 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile target FMS
- name: Compile target depedencies
run: |
make -C .testing \
DO_REGRESSION_TESTS=1 \
Expand Down Expand Up @@ -184,13 +184,14 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile FMS
run: make -C .testing -j build/deps/lib/libFMS.a
- name: Compile dependencies
run: |
make -C .testing -j build/deps/lib/libFMS.a
make -C .testing -j PKG= build/deps/lib/libgsw.a
make -C .testing -j PKG= build/deps/lib/libcvmix.a

- name: Compile MOM6 with aggressive optimization
run: make -C .testing -j build/opt/MOM6
Expand All @@ -217,12 +218,10 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile target FMS
- name: Compile target dependencies
run: |
make -C .testing \
DO_REGRESSION_TESTS=1 \
Expand Down Expand Up @@ -260,13 +259,14 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile FMS
run: make -C .testing -j build/deps/lib/libFMS.a
- name: Compile dependencies
run: |
make -C .testing -j build/deps/lib/libFMS.a
make -C .testing -j PKG= build/deps/lib/libgsw.a
make -C .testing -j PKG= build/deps/lib/libcvmix.a

- name: Compile MOM6 with code coverage
run: make -C .testing -j build/cov/MOM6
Expand Down Expand Up @@ -296,13 +296,14 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup/

- name: Compile FMS
run: make -C .testing -j build/deps/lib/libFMS.a
- name: Compile dependencies
run: |
make -C .testing -j build/deps/lib/libFMS.a
make -C .testing -j PKG= build/deps/lib/libgsw.a
make -C .testing -j PKG= build/deps/lib/libcvmix.a

- name: Compile MOM6 for the GFDL coupled driver
run: make -C .testing -j check_mom6_api_coupled
Expand All @@ -317,8 +318,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand Down Expand Up @@ -350,8 +349,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand All @@ -375,8 +372,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand All @@ -400,8 +395,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand All @@ -425,8 +418,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand Down Expand Up @@ -460,8 +451,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand All @@ -487,8 +476,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand Down Expand Up @@ -522,8 +509,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand Down Expand Up @@ -558,8 +543,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand Down Expand Up @@ -592,8 +575,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand Down Expand Up @@ -638,8 +619,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand Down Expand Up @@ -670,8 +649,6 @@ jobs:

steps:
- uses: actions/checkout@v4
with:
submodules: recursive

- uses: ./.github/actions/ubuntu-setup

Expand Down
Loading