Bump local dependencies#306
Merged
Merged
Conversation
... because we don't longer support it.
Add a KDoc reference to #305 on the `prohibit an entry with empty string` test so the rationale for it being `@Disabled` is discoverable from the source.
Contributor
There was a problem hiding this comment.
Pull request overview
Bumps local Spine dependencies (Validation 419→430, Time 240→241, CoreJvmCompiler 064→065 dogfooding and 063→065 integration, Kotest 6.1.10→6.1.11), bumps the project version to SNAPSHOT.431, removes the now-unneeded patchGeneratedTemplateString workaround (presumably the upstream fix in Validation 430 makes it obsolete), and re-enables several tests that were waiting on those upstream changes. It also introduces a large .claude/ / .agents/ configuration tree for Claude Code agents, skills, hooks, and slash commands.
Changes:
- Bump
validationVersion,Validation,Time,CoreJvmCompiler, andkotestVersion; refreshdocs/dependencies/accordingly. - Remove
PatchGeneratedTemplateString.ktand its invocations incontext/build.gradle.ktsandjvm-runtime/build.gradle.kts; re-enable previously-@Disabledtests inTimestampWhenSpec,TemporalWhenSpec,RequiredRuleITest; deleteExternalConstraintITest. - Add a
.claude/agent/skill/hook/command tree and update.agents/docs to makeversion.gradle.ktsoptional and document new skills.
Reviewed changes
Copilot reviewed 44 out of 44 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| version.gradle.kts | Bumps published snapshot to 431. |
| buildSrc/src/main/kotlin/io/spine/dependency/local/Validation.kt | Bumps Validation to SNAPSHOT.430. |
| buildSrc/src/main/kotlin/io/spine/dependency/local/Time.kt | Bumps Time to SNAPSHOT.241. |
| buildSrc/src/main/kotlin/io/spine/dependency/local/CoreJvmCompiler.kt | Bumps dogfooding/integration versions; fixes KDoc typo. |
| buildSrc/build.gradle.kts | Bumps Kotest to 6.1.11. |
| buildSrc/src/main/kotlin/PatchGeneratedTemplateString.kt | Removes legacy template-string patch task. |
| context/build.gradle.kts, jvm-runtime/build.gradle.kts | Removes call sites of patchGeneratedTemplateString. |
| tests/time/.../TimestampWhenSpec.kt, TemporalWhenSpec.kt | Re-enables time-related test classes. |
| tests/consumer/.../RequiredRuleITest.kt | Re-enables two (required) list tests; expects 2 violations each. |
| tests/consumer/.../Assertions.kt | Adds assertValidationExceptions returning the full violation list. |
| tests/validating/.../RequiredMapWithStringsITest.kt | Adds KDoc explaining why an entry-with-empty-string test stays disabled. |
| tests/validating/.../ExternalConstraintITest.kt | Removes obsolete external-constraint test. |
| docs/dependencies/pom.xml, dependencies.md | Refreshes dependency reports for the new versions. |
| .claude/** (agents, commands, scripts, settings) | New Claude Code configuration: agents, slash commands, and PreToolUse/PostToolUse hooks. |
| .agents/** | Documents new skills; relaxes version-policy to allow repos without version.gradle.kts; KDoc/text tweaks. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
armiol
approved these changes
May 20, 2026
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.
This PR updates dependencies on recently updated CoreJvm Compiler, Time, and Validation.
Other notable changes
(when)option handling.RequiredMapWithStringsITest.configwas pulled.