Fix schema split to respect type boundaries, remove LFS references

andimarek · claude · andimarek · commit 1123cca3b022 · 2026-02-12T07:30:25.000+10:00
- Re-split large-schema-5 at a type boundary so each part contains
  only complete GraphQL type definitions
- Remove Git LFS suggestions from pre-commit hook, CI workflow,
  CONTRIBUTING.md, and README
- Simplify large-schema-5-README.md

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/.githooks/pre-commit b/.githooks/pre-commit
@@ -67,8 +67,7 @@ if [ -n "$LARGE_FILES" ]; then
     echo -e "$LARGE_FILES"
     echo -e "${YELLOW}Please consider one of these options:${NC}"
     echo -e "${YELLOW}  1. Split the file into smaller parts with suffixes .part1, .part2, etc.${NC}"
-    echo -e "${YELLOW}  2. Use Git Large File Storage (Git LFS): https://git-lfs.github.com/${NC}"
-    echo -e "${YELLOW}  3. Remove unnecessary content from the file${NC}"
+    echo -e "${YELLOW}  2. Remove unnecessary content from the file${NC}"
     ERRORS_FOUND=1
 fi
 
diff --git a/.github/workflows/validate-files.yml b/.github/workflows/validate-files.yml
@@ -88,8 +88,7 @@ jobs:
               echo ""
               echo "Please consider one of these options:"
               echo "  1. Split the file into smaller parts with suffixes .part1, .part2, etc."
-              echo "  2. Use Git Large File Storage (Git LFS): https://git-lfs.github.com/"
-              echo "  3. Remove unnecessary content from the file"
+              echo "  2. Remove unnecessary content from the file"
               exit 1
           else
               echo "✓ All files are within the 10MB size limit"
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -38,7 +38,6 @@ The pre-commit hook will automatically check for:
 
 - **Large files**: Files larger than 10MB will be rejected. If you need to commit large files, consider:
   - Splitting them into smaller parts (`.part1`, `.part2`, etc.)
-  - Using [Git Large File Storage (LFS)](https://git-lfs.github.com/)
   - Reducing the file size
 
 To bypass the hooks temporarily (not recommended), use `git commit --no-verify`.
diff --git a/src/test/resources/large-schema-5-README.md b/src/test/resources/large-schema-5-README.md
@@ -1,33 +1,14 @@
-# Large Schema File - Split for Windows Compatibility
+# large-schema-5 — Split File
 
-The file `large-schema-5.graphqls` (11.3 MB) has been split into multiple parts to comply with the 10MB file size limit.
+`large-schema-5.graphqls` (11.3 MB) exceeds the 10 MB file-size limit and has been split into two parts.
+The split is at a type boundary so each part contains only complete type definitions.
 
-## Parts
+## Reassembly
 
-- `large-schema-5.graphqls.part1` - First part (6.2 MB)
-- `large-schema-5.graphqls.part2` - Second part (4.7 MB)
-
-## Reassembly Instructions
-
-To reassemble the original file, use one of the following methods:
-
-### On Linux/macOS:
 ```bash
+# Linux / macOS
 cat large-schema-5.graphqls.part1 large-schema-5.graphqls.part2 > large-schema-5.graphqls
-```
 
-### On Windows (PowerShell):
-```powershell
+# Windows (PowerShell)
 Get-Content large-schema-5.graphqls.part1, large-schema-5.graphqls.part2 | Set-Content large-schema-5.graphqls
 ```
-
-### On Windows (Command Prompt):
-```cmd
-copy /b large-schema-5.graphqls.part1+large-schema-5.graphqls.part2 large-schema-5.graphqls
-```
-
-## For Test Code
-
-If test code references `large-schema-5.graphqls`, update it to either:
-1. Reassemble the file programmatically before use
-2. Update the code to read from both parts sequentially
diff --git a/src/test/resources/large-schema-5.graphqls.part1 b/src/test/resources/large-schema-5.graphqls.part1
@@ -106167,6 +106167,3 @@ type Object7024 implements Interface208 @Directive29(argument64 : "stringValue98
   field31303: String @Directive42(argument112 : true) @Directive51
 }
 
-type Object7025 implements Interface208 @Directive29(argument64 : "stringValue98430", argument65 : false, argument66 : false, argument67 : false) @Directive31(argument69 : "stringValue98427") @Directive4(argument3 : ["stringValue98428", "stringValue98429"]) @Directive42(argument112 : true) {
-  field27360: Enum1505 @Directive42(argument112 : true) @Directive51
-  field27361: Object5945 @Directive42(argument112 : true) @Directive51
diff --git a/src/test/resources/large-schema-5.graphqls.part2 b/src/test/resources/large-schema-5.graphqls.part2
@@ -1,3 +1,6 @@
+type Object7025 implements Interface208 @Directive29(argument64 : "stringValue98430", argument65 : false, argument66 : false, argument67 : false) @Directive31(argument69 : "stringValue98427") @Directive4(argument3 : ["stringValue98428", "stringValue98429"]) @Directive42(argument112 : true) {
+  field27360: Enum1505 @Directive42(argument112 : true) @Directive51
+  field27361: Object5945 @Directive42(argument112 : true) @Directive51
   field27364: Object5946 @Directive42(argument112 : true) @Directive51
   field31296: Object7026 @Directive42(argument112 : true) @Directive51
   field31303: String @Directive42(argument112 : true) @Directive51

Original file line number	Diff line number	Diff line change
`@@ -106167,6 +106167,3 @@ type Object7024 implements Interface208 @Directive29(argument64 : "stringValue98`
`106167`	`106167`	`field31303: String @Directive42(argument112 : true) @Directive51`
`106168`	`106168`	`}`
`106169`	`106169`
`106170`		`-type Object7025 implements Interface208 @Directive29(argument64 : "stringValue98430", argument65 : false, argument66 : false, argument67 : false) @Directive31(argument69 : "stringValue98427") @Directive4(argument3 : ["stringValue98428", "stringValue98429"]) @Directive42(argument112 : true) {`
`106171`		`- field27360: Enum1505 @Directive42(argument112 : true) @Directive51`
`106172`		`- field27361: Object5945 @Directive42(argument112 : true) @Directive51`