Skip to content

Bootstrap module analysis#305

Merged
deweyjose merged 4 commits intomainfrom
cursor/bootstrap-module-analysis-6e49
Feb 28, 2026
Merged

Bootstrap module analysis#305
deweyjose merged 4 commits intomainfrom
cursor/bootstrap-module-analysis-6e49

Conversation

@deweyjose
Copy link
Owner

Remove the bootstrap module and vendor the GeneratedCodeGenConfigBuilder to simplify the project structure and reduce build complexity.


Open in Web Open in Cursor 

Co-authored-by: deweyjose <deweyjose@users.noreply.github.com>
@cursor
Copy link
Contributor

cursor bot commented Feb 26, 2026

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
Learn more about Cursor Agents

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

Code Coverage Report

Overall Project 88.56% 🍏
Files changed 100% 🍏

File Coverage
IntrospectionRequest.java 100% 🍏
CodeGenConfigBuilder.java 100% 🍏
TypeMappingService.java 96.55% 🍏
SchemaTransformationService.java 95.79% 🍏
SchemaManifestService.java 93.43% 🍏
RemoteSchemaService.java 93.38% 🍏
SchemaFileService.java 91.93% 🍏
CodegenExecutor.java 84.35% 🍏
ParameterMap.java 71.43% 🍏
Logger.java 15.38% 🍏
Constants.java 0% 🍏
Codegen.java 0% 🍏

@deweyjose deweyjose marked this pull request as ready for review February 26, 2026 00:25
@deweyjose
Copy link
Owner Author

@cursoragent this is great but one bit - I don't think we need to use the "Generated" prefixes for the builder now. Can you look at renaming the class and package it with the other source files?

@cursor
Copy link
Contributor

cursor bot commented Feb 26, 2026

Great call — I went ahead and did exactly that.

✅ Renamed and relocated

  • Old: io.github.deweyjose.codegen.generated.GeneratedCodeGenConfigBuilder
  • New: io.github.deweyjose.graphqlcodegen.CodeGenConfigBuilder

File moved to:

  • graphqlcodegen-maven-plugin/src/main/java/io/github/deweyjose/graphqlcodegen/CodeGenConfigBuilder.java

✅ Updated usages/docs

  • CodegenExecutor now uses new CodeGenConfigBuilder()
  • Removed old generated-package/class reference
  • Updated README.md + Agents.md references to the new class/location

✅ Validation

  • Ran: ./mvnw -pl graphqlcodegen-maven-plugin test
  • Result: PASS (47 tests, 0 failures)

Pushed in commit:

  • 060de01

Open in Web Open in Cursor 

cursoragent and others added 3 commits February 26, 2026 00:31
Co-authored-by: deweyjose <deweyjose@users.noreply.github.com>
Co-authored-by: deweyjose <deweyjose@users.noreply.github.com>
Co-authored-by: deweyjose <deweyjose@users.noreply.github.com>
@deweyjose deweyjose merged commit adcb75b into main Feb 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants