Skip to content

Rebuild with refreshed dependencies - #14

Merged
alexander-yevsyukov merged 17 commits into
masterfrom
cascade-refresh-219fef6
Oct 7, 2026
Merged

alexander-yevsyukov merged 17 commits into
masterfrom
cascade-refresh-219fef6

Conversation

@alexander-yevsyukov

@alexander-yevsyukov alexander-yevsyukov commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Catches reflect up to the current shared config baseline (config@219fef6d).
The repository last pulled config in June 2025, about 900 commits behind, so
this is a migration rather than a routine refresh. Part of the straggler
catch-up driven from summit (docs/rollout/rebuild.md).

The version moves to 2.0.0-SNAPSHOT.201, and the repository switches to the
CodeMatters copyright profile.

Commits

  • Update shared agent tooling and config (./config/pull): floats the shared
    tooling and the config-distributed build files: Gradle 9.7.1, the current
    buildSrc, workflows, and the .agents/shared submodule.
  • Bump version -> 2.0.0-SNAPSHOT.201: also migrates version.gradle.kts
    from the deprecated by extra(...) to extra.set(...).
  • Select CodeMatters (c) profile in IDEA settings and Update copyright
    headers to CodeMatters profile
    : re-stamps the 85 project-owned sources,
    headers only. A second Bump version commit re-stamps the header of
    version.gradle.kts; the version policy requires that subject for any
    change to the file.
  • Remove stale buildSrc files: ./config/pull overlays buildSrc with
    cp -R, which never deletes files config stopped shipping. 20 such files no
    longer compiled (13 errors). Each one was checked to be an unedited copy of a
    file config once shipped and has since deleted.
  • Adapt to refreshed dependencies (two commits; please review these with
    care):
    • drop the dokkaJar { java = true } block, which config removed;
    • force Coroutines.bom, since TestLib.lib still requests 1.10.2 against
      the 1.11.0 baseline;
    • exclude the given.unloaded test fixtures with Test.exclude(...) instead
      of filter.excludeTestsMatching(...). Under Gradle 9.7.1 the filter still
      lets test discovery load those classes, which broke the
      PackageAnnotationLookupSpec case that needs them unloaded;
    • implement PackageGraph.asNetwork(), which Guava 33.7.1 made required for
      Graph, by delegating to the wrapped ImmutableGraph;
    • clear the new Error Prone, PMD, and Kotlin compiler warnings in
      PackageInfo.equals and ClassExtsSpec.
  • Describe the Reflect project in docs/project.md: fills in the template
    the pull seeded; a follow-up commit fixes its guideline link and wording.
  • Test PackageGraph.asNetwork(): adds coverage for the new delegate.
  • Update dependency reports: docs/dependencies/ regenerated by the build.

Verification

  • ./gradlew clean build dokkaGenerate passes; 103/103 tests.
  • A build without the build cache shows no compiler, Error Prone, PMD, or detekt
    warnings in project sources. The only Gradle deprecations come from the Gradle
    Doctor and detekt plugins.
  • pre-pr passed at this PR's HEAD, with spine-code-review, kotlin-engineer,
    and review-docs all approving.
  • 2.0.0-SNAPSHOT.201 is published to Maven Local.

🤖 Generated with Claude Code

alexander-yevsyukov and others added 13 commits October 7, 2026 17:10
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Re-stamp the copyright header with the CodeMatters profile; the version is unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
`./config/pull` overlays `buildSrc` with `cp -R`, which never deletes files
that `config` stopped shipping. This repository last pulled in June 2025, so
it kept 20 such files. They no longer compiled against the rest of `buildSrc`
(for example, `Runtime.kt` redeclared `Cli`, and `ArtifactVersion.kt` used the
retired `ModelCompiler`), failing `:buildSrc:compileKotlin` with 13 errors.

Each removed file is a byte-for-byte copy of a version `config` once shipped
at the same path and has since deleted. This repository has no `buildSrc`
files of its own, so nothing it owns is affected.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Drop the `dokkaJar { java = true }` block: `config` removed the option
  together with the `dokka-for-java` plugin.
- Force `Coroutines.bom`: `TestLib.lib` still requests 1.10.2 against the
  1.11.0 baseline.
- Exclude the `given.unloaded` fixtures from the test-discovery candidates.
  Under Gradle 9.7.1 the `filter.excludeTestsMatching` filter still lets the
  discovery load those classes, which broke the `PackageAnnotationLookupSpec`
  case that needs them unloaded.
- Implement `PackageGraph.asNetwork()`, which Guava 33.7.1 made required
  for `Graph`, by delegating to the wrapped `ImmutableGraph`.
- Address the new warnings: use a pattern-matching `instanceof` in
  `PackageInfo.equals`, and `Float::class.javaObjectType` in place of
  `java.lang.Float::class.java` in `ClassExtsSpec`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Fold `PackageInfo.equals` into a single boolean expression: with the
pattern-matching `instanceof`, PMD 7.27 reports `SimplifyBooleanReturns`
on the early-return form.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- `docs/project.md`: resolve the guideline link relative to `docs/`, name
  `PackageGraph` and `PackageAnnotationLookup` instead of ambiguous pronouns,
  move the test-discovery note into its own bullet, and attribute JSpecify to
  the `jvm-module` convention along with Guava and Protobuf.
- `build.gradle.kts`: say which class files `Test.exclude` drops and when it
  applies, and note when the `Coroutines.bom` force can go.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-07T17:05:09.358291Z f4e2834 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approved

The repository-owned changes are consistent, behavior-neutral, and accompanied by a successful clean build and documentation generation.

0 open findings

What changed in this PR

Rebuilds Spine Reflect against refreshed shared tooling and dependencies without changing library behavior.

Changes:

  • Bumps the publication version to 2.0.0-SNAPSHOT.201.
  • Refreshes shared Gradle, CI, dependency, and agent configuration.
  • Standardizes source-file copyright headers and adds project architecture documentation.
File Description
version.gradle.kts Bumps the publication version.
settings.gradle.kts Updates the copyright header.
gradle/​wrapper/​gradle-wrapper.properties Updates Gradle wrapper configuration.
docs/​project.md Documents project purpose and architecture.
CLAUDE.md Refreshes shared agent guidance.
.gitmodules Adds the shared agents submodule.
.codecov.yml Refreshes coverage configuration.
.junie/​guidelines.md Updates shared Junie guidance.
.idea/​live-templates/​User.xml Corrects the date template format.
.idea/​live-templates/​README.md Corrects documentation wording.
.idea/​kotlinc.xml Removes obsolete IDE compiler settings.
.idea/​inspectionProfiles/​Project_Default.xml Normalizes the file ending.
.idea/​copyright/​profiles_settings.xml Selects the CodeMatters profile.
.idea/​copyright/​CodeMatters_Open_Source.xml Adds the CodeMatters copyright profile.
.github/​workflows/​remove-obsolete-artifacts-from-packages.yaml Refreshes workflow names and checkout action.
.github/​workflows/​gradle-wrapper-validation.yml Removes the standalone wrapper workflow.
.github/​workflows/​build-on-windows.yml Refreshes Windows CI tooling.
.agents/​version-policy.md Removes superseded local guidance.
.agents/​testing.md Removes superseded local guidance.
.agents/​safety-rules.md Removes superseded local guidance.
.agents/​running-builds.md Removes superseded local guidance.
.agents/​refactoring-guidelines.md Removes superseded local guidance.
.agents/​quick-reference-card.md Removes superseded local guidance.
.agents/​project-structure-expectations.md Removes superseded local guidance.
.agents/​documentation-tasks.md Removes superseded local guidance.
.agents/​documentation-guidelines.md Removes superseded local guidance.
.agents/​common-tasks.md Removes superseded local guidance.
.agents/​coding-guidelines.md Removes superseded local guidance.
.agents/​advanced-safety-rules.md Removes superseded local guidance.
.agents/​_TOC.md Removes the obsolete local index.
src/​main/​kotlin/​io/​spine/​reflect/​ThrowableStackGetter.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​StackWalkerStackGetter.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​StackGetter.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​PackageExts.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​PackageAnnotationLookup.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​JvmPackages.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​Factory.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​ClassExts.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​CallerFinder.kt Standardizes the copyright header.
src/​main/​kotlin/​io/​spine/​reflect/​AnnotatedPackages.kt Standardizes the copyright header.
src/​main/​java/​io/​spine/​reflect/​Types.java Standardizes the copyright header.
src/​main/​java/​io/​spine/​reflect/​package-info.java Standardizes the copyright header.
src/​main/​java/​io/​spine/​reflect/​J2Kt.java Standardizes the copyright header.
src/​main/​java/​io/​spine/​reflect/​IsDirectParent.java Standardizes the copyright header.
src/​main/​java/​io/​spine/​reflect/​Invokables.java Standardizes the copyright header.
src/​main/​java/​io/​spine/​reflect/​GenericTypeIndex.java Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​ThrowableStackGetterSpec.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​StackWalkerStackGetterSpec.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​PackageAnnotationLookupSpec.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​J2KtSpec.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​FactorySpec.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​CallerFinderSpec.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​AbstractStackGetterSpec.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​given/​UserCode.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​given/​MethodHolder.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​given/​MemoizingJvmPackages.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​given/​LoggerCode.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​given/​LogContext.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​given/​ClassWithCompanion.kt Standardizes the copyright header.
src/​test/​kotlin/​io/​spine/​reflect/​given/​CallingElvis.kt Standardizes the copyright header.
src/​test/​kotlin/​given/​reflect/​KotlinClass.kt Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​TypesTest.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​PackageInfoTest.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​PackageGraphFilterTest.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​IsDirectParentTest.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​InvokablesTest.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​GenericTypeIndexTest.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​TypesTestEnv.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​TestAnnotationContainer.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​TestAnnotation.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​RepeatableTestAnnotation.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​package-info.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​MethodsTestEnv.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​InapplicableTestAnnotation.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​ConstructorsTestEnv.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​nested1/​package-info.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​nested1/​Nested1.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​nested1/​nested2/​package-info.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​unloaded/​nested1/​UnloadedNested1.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​unloaded/​nested1/​package-info.java Standardizes the copyright header.
src/​test/​java/​io/​spine/​reflect/​given/​unloaded/​nested1/​nested2/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​WithParametersJava.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​JavaClass.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​annotation/​ValueAnnotation.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​annotation/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch2/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch2/​Branch2Class.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch2/​lorem/​ipsum/​Sub3Class.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch2/​lorem/​ipsum/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch1/​foo/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch1/​foo/​FooClass.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch1/​foo/​sub1/​Sub1Class.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch1/​foo/​sub1/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch1/​foo/​sub2/​Sub2Class.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch1/​foo/​sub2/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch1/​bar/​package-info.java Standardizes the copyright header.
src/​test/​java/​given/​reflect/​root/​branch1/​bar/​LastVisitor.java Standardizes the copyright header.
buildSrc/​src/​main/​kotlin/​module.gradle.kts Adds the project convention-script template.
buildSrc/​src/​main/​kotlin/​test-module.gradle.kts Adds shared test-module configuration.
buildSrc/​src/​main/​kotlin/​BuildSettings.kt Refreshes shared build tooling.
buildSrc/​src/​main/​kotlin/​dokka-for-kotlin.gradle.kts Removes obsolete Dokka convention.
buildSrc/​src/​main/​kotlin/​dokka-for-java.gradle.kts Removes obsolete Dokka convention.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​TaskName.kt Refreshes shared build tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​StringExtensions.kt Refreshes shared build tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​SpineTaskGroup.kt Adds the shared Spine task group.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​RunBuild.kt Refreshes shared build tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​Clean.kt Refreshes shared build tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​Build.kt Refreshes shared build tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​base/​Tasks.kt Refreshes shared task tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​dokka/​TaskContainerExtensions.kt Removes obsolete Dokka helper.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​git/​Branch.kt Refreshes shared Git tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​git/​UserInfo.kt Refreshes shared Git tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​github/​pages/​AuthorEmail.kt Refreshes shared Pages tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​javac/​Javac.kt Refreshes Java compiler tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​javadoc/​Encoding.kt Refreshes Javadoc tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​javadoc/​JavadocTag.kt Refreshes Javadoc tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​publish/​CloudRepo.kt Refreshes publishing tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​publish/​PublishingRepos.kt Refreshes publishing tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​repo/​Credentials.kt Refreshes repository tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​report/​coverage/​FileExtension.kt Removes obsolete coverage helper.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​report/​coverage/​TaskName.kt Removes obsolete coverage task names.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​report/​license/​Configuration.kt Refreshes license reporting.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​report/​license/​MarkdownReportRenderer.kt Refreshes license reporting.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​report/​license/​ProjectDependencies.kt Refreshes license reporting.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​report/​pom/​MarkupExtensions.kt Refreshes POM reporting.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​report/​pom/​ModuleDependency.kt Refreshes POM reporting.
buildSrc/​src/​main/​kotlin/​io/​spine/​gradle/​report/​pom/​SpineLicense.kt Refreshes POM license metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​docs/​MarkdownDocument.kt Refreshes documentation tooling.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​build/​CheckStyle.kt Refreshes build dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​build/​FindBugs.kt Refreshes build dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​build/​LicenseReport.kt Removes obsolete dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​build/​OsDetector.kt Refreshes build dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​kotlinx/​AtomicFu.kt Adds AtomicFU dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​kotlinx/​DateTime.kt Adds Kotlin date-time metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​ApacheHttp.kt Refreshes dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​AppEngine.kt Removes obsolete dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​BouncyCastle.kt Refreshes dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​JavaJwt.kt Refreshes dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​JavaPoet.kt Refreshes dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​JavaX.kt Refreshes dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​JetBrainsAnnotations.kt Adds annotations dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​Klaxon.kt Refreshes dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​KotlinX.kt Refreshes dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​PalantirJavaFormat.kt Adds formatter dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​lib/​PerfMark.kt Adds PerfMark dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​local/​DokkaTools.kt Adds Dokka tooling coordinates.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​local/​ModelCompiler.kt Removes obsolete local coordinates.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​local/​Text.kt Removes obsolete local coordinates.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​storage/​H2.kt Adds H2 dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​storage/​Hikari.kt Adds HikariCP dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​storage/​HsqlDb.kt Adds HSQLDB dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​storage/​MySql.kt Adds MySQL dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​storage/​PostgreSql.kt Adds PostgreSQL dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​storage/​QueryDsl.kt Adds QueryDSL dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​test/​Hamcrest.kt Refreshes test dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​test/​OpenTest4J.kt Refreshes test dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​test/​SystemLambda.kt Refreshes test dependency metadata.
buildSrc/​src/​main/​kotlin/​io/​spine/​dependency/​test/​TestKitTruth.kt Refreshes test dependency metadata.
buildSrc/​src/​main/​resources/​dokka/​styles/​custom-styles.css Standardizes the copyright header.
buildSrc/​src/​test/​kotlin/​io/​spine/​dependency/​lib/​KotlinSpec.kt Adds dependency metadata tests.
buildSrc/​src/​test/​kotlin/​io/​spine/​dependency/​lib/​ProtobufSpec.kt Adds dependency metadata tests.
buildSrc/​src/​test/​kotlin/​io/​spine/​gradle/​fs/​SpineTempDirSpec.kt Adds temporary-directory tests.
buildSrc/​settings.gradle.kts Refreshes shared build settings.
buildSrc/​quality/​detekt-config.yml Adds shared Detekt configuration.
buildSrc/​quality/​checkstyle-suppressions.xml Adds generated-source suppressions.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@codecov

codecov Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.72%. Comparing base (980720d) to head (791c16c).

Additional details and impacted files
@@             Coverage Diff              @@
##             master      #14      +/-   ##
============================================
- Coverage     86.84%   86.72%   -0.13%     
- Complexity      127      129       +2     
============================================
  Files            17       17              
  Lines           479      467      -12     
  Branches         62       62              
============================================
- Hits            416      405      -11     
+ Misses           49       48       -1     
  Partials         14       14              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

alexander-yevsyukov and others added 4 commits October 7, 2026 18:15
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The `jvm-module` convention already applies the Dokka and `project-report`
plugins, adds `JSpecify.annotations`, and calls `LicenseReporter.generateReportIn`,
`CheckStyleConfig.applyTo`, and `JavadocConfig.applyTo`. Drop the repeats, the
second `repositories.standardToSpineSdk()` call, which registered every
repository twice, and the `productionModules` Dokka block, a no-op in this
single-module project. Also drop the `@file:Suppress` that no longer applies.

Verified: the applied plugins, tasks, test, Javadoc and Checkstyle settings,
dependency graphs, and the published POM are unchanged; only the duplicate
repository entries are gone.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@alexander-yevsyukov
alexander-yevsyukov merged commit db12351 into master Oct 7, 2026
8 of 9 checks passed
@alexander-yevsyukov
alexander-yevsyukov deleted the cascade-refresh-219fef6 branch October 7, 2026 18:11
alexander-yevsyukov added a commit to SpineEventEngine/summit that referenced this pull request Oct 8, 2026
`reflect`, `testlib`, and `money` are caught up (SpineEventEngine/reflect#14,
SpineEventEngine/testlib#14, SpineEventEngine/money#32), so their first
`config/pull` is no longer a migration. Excluding them by default would let
them fall behind again. Drop the default exclusion and update the rollout
text: the single-repository wave example no longer needs `--include`, and the
`retarget` prerequisite is stated for every repository.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
alexander-yevsyukov added a commit to SpineEventEngine/summit that referenced this pull request Oct 9, 2026
* Fix the `require_java` crash when `JAVA_HOME` is unset

Under `set -euo pipefail`, probing `${JAVA_HOME:-/nonexistent}/bin/java`
failed the pipeline and aborted the script with exit 127 and no message,
before the JDK 17 fallback could run. Tolerate the failed probe so the
fallback runs and the existing "no JDK found" error stays the only failure.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Add `settings.gradle.kts` naming the root project `summit`

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Describe `documentation` as the spine.io documentation area

`documentation` is the documentation area of the spine.io site, not only
a Spine 1.x project. Correct the note in the graph template and the reason
`plan` prints for excluding the repository, and regenerate
`docs/dependency-graph.md`, which also brings it in line with the template's
current formatting.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Add the Gradle wrapper

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update the IDEA inspection profile

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Add the `tool-base` split migration plan

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update cascade state

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update submodule pointers

Record the commits currently checked out:

- `agents`, `base-libraries`, `config`, `core-jvm-compiler`, `gcloud-jvm`,
  and `jdbc-storage`: their `master` tips.
- `core-jvm`: a commit on its `claude/*` branches, not yet on `master`.
- `reflect`: the head of the open straggler catch-up PR
  (SpineEventEngine/reflect#14, branch `cascade-refresh-219fef6`).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Bump Gradle -> `9.8.1`

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update cascade state

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Archive the completed wave's task files

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Record detekt's `Configuration.setVisible` deprecation as third-party

Gradle 9.8 reports `Configuration.setVisible(boolean)` as deprecated. The call
comes from the detekt Gradle plugin, like the `ReportingExtension.file` warning
already listed, so no Spine repository can fix it in its own code.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Keep `cascade plan` from overwriting an archived wave

`plan` wrote the new manifest through `manifest_path`, which falls back to
`archive/` when no active manifest exists. Planning a wave whose slug matched
an archived one therefore overwrote the archived manifest. Successive
single-repo waves against the same `config` commit share a slug, so this hit
the straggler catch-up when `testlib` followed `reflect`.

Write the manifest to the top level explicitly, refuse a slug whose wave is
already archived, and add `--slug-suffix <word>` to give such a wave its own
name. Document the one-repository-at-a-time recipe in `rebuild.md`.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update cascade state

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update the `config` submodule pointer

Record `config` at the merge of SpineEventEngine/config#784, which bumps Gradle to 9.8.1 and closes the `reflect` catch-up wave.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update cascade state

* Update cascade state

* Archive the completed wave's task files

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update the `testlib` submodule pointer

Record `testlib` at the merge of SpineEventEngine/testlib#14, its straggler catch-up.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update cascade state

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update cascade state

* Update the `config` submodule pointer

Record `config` at the merge of SpineEventEngine/config#785, which closes the `testlib` catch-up wave.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update cascade state

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update shared agent tooling and config (./config/pull)

Pull `config@3409e7dd`. Keep `summit`'s own `.claude/settings.json`: the shared
copy would drop its `git push --force` deny rules, its `git commit` and
`./cascade` allowances, and the `git tag` ask rule.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update submodule pointers

Record the `master` tips of `agents`, `core-jvm`, `core-jvm-compiler`,
`delivery-server`, `dokka-tools`, `money` (with the merged straggler catch-up,
SpineEventEngine/money#32), and `reflect` (moved off its merged wave branch).

`tool-base` stays as it is: its checkout is on a feature branch of a separate task.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update cascade state

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Archive the completed wave's task files

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update the `tool-base` submodule pointer

Record `tool-base` at the merge of SpineEventEngine/tool-base#193, which stops `GeneratedSourcePlugin` from invoking `Task.project` at execution time.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Show repeated `--exclude` in the single-repository wave example

Each `--exclude` takes one repository, so a space-separated list after a single `--exclude` stops `plan` with its usage message. Repeat the option in the example and say so in the prose; also note that the shared-name case applies to `refresh` waves.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Stop excluding the former stragglers from `refresh` waves

`reflect`, `testlib`, and `money` are caught up (SpineEventEngine/reflect#14,
SpineEventEngine/testlib#14, SpineEventEngine/money#32), so their first
`config/pull` is no longer a migration. Excluding them by default would let
them fall behind again. Drop the default exclusion and update the rollout
text: the single-repository wave example no longer needs `--include`, and the
`retarget` prerequisite is stated for every repository.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Update submodule pointers

Record the current `master` tips of `core-jvm` and `core-jvm-compiler`, which advanced after the previous pointer update.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants