Lift 4.0 #2043
Open
Lift 4.0 #2043
Conversation
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
Updates sbt, scala, jetty, javax.servlet -> jakarta.servlet, and fixes some of the deprecated scala syntax in preperation for moving to scala 3. Fixed: type Traversable in package scala is deprecated (since 2.13.0): Use Iterable instead of Traversable object JavaConverters in package collection is deprecated (since 2.13.0): Use scala.jdk.CollectionConverters instead view bounds are deprecated; use an implicit parameter instead. Implicit definition should have explicit type Adding a number and a String is deprecated. Use the string interpolation s"$num$str" Dropped EnumWithDescription which seems to not be used and contained a cyclic reference that no longer compiles. Removed the cross compiling for now and dropped the persistence library from build.sbt and did not attempt to update it.
Remove the persistence components
Remove squeryl and JSON scala combinators parser
Build and Readme Updates for Lift 4.x
Lift-4: remove simple warnings
Configurable LAPinger.service
* feat(lift-actor)!: support scala 3 * feat(lift-webkit)!: remove comet bit that depended on java interop
- rename `inline` to `inlineParser` to avoid the keyword collision - fix indentation - change postfix `*` to `.*` - removed junit specific bits b/c tests run fine without them
* feat!: migrate from lift-json to json4s * chore: mark noTime test as pending until fixed not sure why my migration to json4s would have caused a failure here
* fix(lift-util): bring in correct scala3 compiler dependency * feat(lift-util)!: scala 3.3.6 compatibility fixes - Fix indentation to comply with Scala 3's stricter rules - Add explicit type bounds where Scala 3 type inference requires them - Fix implicit conversions for Scala 3 compatibility - Remove SourceInfo.scala (relied on Scala 2 reflection) - Remove deprecated implicit conversions in TimeHelpers BREAKING CHANGE: Removes SourceInfo trait and related functionality that relied on Scala 2 reflection APIs not available in Scala 3. * fix(lift-util): resolve scala 3.3.6 compilation errors - Fix StringHelpers tuple addition in foldLeft - Add public hasMoreInput method to PCDataXmlParser to work around Scala 3 protected access restrictions - Add case keyword to refutable pattern match in Props.scala 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * feat(lift-util): work on getting tests cross-building * fix(lift-util): get tests passing for scala 3 again --------- Co-authored-by: Claude <noreply@anthropic.com>
Reduce Scala 3 compilation errors from 354 to 2 (99.4% reduction). ## Major Changes ### Scala 3 Syntax & Type System - Fix FactoryMaker vend() call syntax (vend()() vs vend.apply()) - Add explicit types to PartialFunction pattern matches - Fix lambda parameter syntax (require parentheses with type annotations) - Fix union type handling for Box[NodeSeq].openOr with explicit matching - Add explicit return types for recursive/overloaded methods - Fix indentation issues causing scope problems in Scala 3 ### Visibility & Access Modifiers - Change private[http] to public for cross-package access in Scala 3: - LiftRules: lockedSecurityRules, notFoundOrIgnore, dispatchTable - LiftRules: snippet, bootFinished, withMimeHeaders - Scala 3 has stricter rules for package-private access ### Version-Specific Code - Create scala_3/NamedCometDispatcher.scala without parallel collections - Add scala.jdk.CollectionConverters import for Scala 3 compatibility ### Type Safety Improvements - Fix Box[String] vs String comparisons using .contains() - Add explicit type annotations to createSnapshot methods - Fix toString return type declarations ### Code Structure - Fix case block brace placement for proper scoping - Correct indentation in match expressions and for-comprehensions - Fix lambda syntax in dropWhile, filter, and CSS selector transforms ## Breaking Changes - Some private[http] members are now public - NamedCometDispatcher.scala uses sequential iteration instead of parallel ## Known Issues (2 remaining errors) - Wizard.scala Field trait shadowing issue - Scala 3 doesn't allow nested traits to shadow parent traits with the same name. Requires refactoring to rename Wizard.Screen.Field or create Scala 3-specific version. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Scala 2 and Scala 3 type inference these slightly differently. Being explicit about the type here should ensure that the code cross-compiles successfully.
This doesn't compile correctly in Scala 3
Migrate lift-webkit test suite from Scala 2.13 to Scala 3.3.6. This
resolves all 397+ compilation errors by updating test code to work with
specs2 5.x and Scala 3 requirements.
Key changes:
- Update specs2 Scope usage: Move from org.specs2.specification.Scope
to org.specs2.execute.Scope and change instantiation syntax
- Fix Mockito integration: Replace specs2.mock.Mockito with
scalatestplus.mockito.MockitoSugar and update stubbing syntax
- Handle BeforeEach.before return type change: Wrap side effects in
step {} blocks to return Fragments
- Fix protected method access: Create public wrapper methods for testing
- Update matcher syntax for specs2 5.x: have length -> haveLength
- Fix lambda type inference: Add explicit parameter types
- Serialize JSON to bytes for mock requests using compact/render
- Replace MatchResult with Success/Failure pattern
- Add explicit Unit returns in step blocks
- Add success markers in multi-assertion test blocks
- Fix type ascriptions for Box types in eventually matchers
All test files now compile successfully with only deprecation warnings
(Manifest -> ClassTag transitions) remaining.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Create separate test files for Scala 2.13 and Scala 3 to handle differences between specs2 4.x and 5.x APIs. Changes: - Move 6 test files with API differences to scala-2.13/ and scala-3/ - Keep remaining 37 test files in shared scala/ directory - Tests now compile successfully in both Scala 2.13.16 and 3.3.6 Scala 2.13 (specs2 4.x) uses: - org.specs2.specification.Scope - org.specs2.mock.Mockito with "returns" syntax - result() method for custom matchers Scala 3 (specs2 5.x) uses: - org.specs2.execute.Scope - org.mockito.Mockito with when().thenReturn() syntax - Success/Failure for custom matchers Files with version-specific implementations: - SpecContextHelpers.scala (Scope location) - LiftMergeSpec.scala (Mockito syntax) - ReqSpec.scala (Scope + Mockito) - LiftJavaScriptSpec.scala (Scope location) - OfflineRequestSnapshotSpec.scala (Mockito syntax) - XMLApiSpec.scala (Matcher result format) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…3.3.6
Fixes critical test failures where all LiftMergeSpec tests were failing with
"IllegalStateException: Attempted to use resource bundles outside of an
initialized S scope".
Root Cause:
The test helper traits (WithRules, WithLiftContext) were using the Scope
pattern with empty blocks that caused ThreadLocal state to be set and
immediately cleared BEFORE test execution. This left tests without proper
Lift context (LiftRules and S scope).
The original trait-based approach relied on specs2's Around pattern which
wrapped test execution, but the migration to Scope for specs2 5.x broke
this - Scope only runs code during construction, not during test execution.
Solution:
Migrated from trait-based initialization to explicit wrapper functions
following specs2 5.x best practices for Scala 3 compatibility:
- Replaced trait-based helpers with SpecContextHelpers object
- Added withLiftRules(rules)(test) - wraps test with LiftRules context
- Added withLiftContext(rules, session)(test) - wraps test with both
LiftRules and S scope
- Properly manages ThreadLocal lifecycle during test execution using
doWith {} blocks that contain the actual test code
Changes:
- Refactored SpecContextHelpers.scala (Scala 2.13 & 3.3.6 versions)
- Updated LiftMergeSpec.scala to use new wrapper functions
- Updated HtmlFixerSpec.scala to use new wrapper functions
Results:
- All 13 LiftMergeSpec tests now pass in Scala 2.13.16 ✅
- All 13 LiftMergeSpec tests now pass in Scala 3.3.6 ✅
- Cross-version compatible test infrastructure
- No reliance on deprecated DelayedInit
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Fixes test isolation failure in LiftSessionSpec where the second test was
seeing state from the first test, causing:
Vector(1, 2, ..., 20, 1, 1) != Vector(1, 1)
Root Cause:
The BeforeEach trait's `before` method has different API requirements
between specs2 4.x (Scala 2.13) and specs2 5.x (Scala 3):
- specs2 4.x: before method executes action directly
- specs2 5.x: before method must wrap action in step()
The shared implementation used `step {}` which worked for Scala 3 but
failed silently on Scala 2.13 - the cleanup never ran, causing shared
mutable state to leak between tests.
Solution:
Split LiftSessionSpec into version-specific implementations:
- Scala 2.13: `override def before = { receivedMessages = Vector[Int]() }`
- Scala 3.3.6: `override def before = step { receivedMessages = Vector[Int]() }`
Results:
- All 4 LiftSessionSpec tests now pass in Scala 2.13.16 ✅
- All 4 LiftSessionSpec tests now pass in Scala 3.3.6 ✅
- Proper test isolation with no state leaking between examples
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
Fixes two test failures in SnippetSpec: Scala 2.13.16 failures: - "Correctly process not lift:designer_friendly" (line 84) - "reflect only the last pushed values with S.currentAttrs" (line 450) Scala 3.3.6 failures: - "reflect only the last pushed values with S.currentAttrs" (line 450) Root Causes: 1. XML Comparison Using Wrong Matcher (line 84): Used strict equality (===) instead of XML-aware matcher (==/). The === operator performs object equality which is sensitive to whitespace, attribute ordering, and formatting differences in XML. All other XML tests in the file correctly use the ==/ matcher. 2. Nested Assertion Result Propagation (line 450): Test had assertions only at the innermost nesting level with no assertions as the nested S.withAttrs blocks unwound. This caused the test result to not propagate correctly through the nested function calls, particularly in specs2 4.x (Scala 2.13). Solutions: 1. Changed line 84 from `ret === xml` to `ret must ==/(xml)` to use XML-aware comparison that normalizes XML before matching. 2. Added assertions at each unwinding level of the nested S.withAttrs blocks (lines 448, 451-452, 455) to ensure proper result propagation. This follows the pattern used by the passing test "properly reflect the full snippet stack with S.attrs" (lines 409-435). Results: - All 23 SnippetSpec tests now pass in Scala 2.13.16 ✅ - All 23 SnippetSpec tests now pass in Scala 3.3.6 ✅ - 2 tests remain pending (intentionally marked as such in the code) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The Box class defines its own === operator that returns Boolean for value comparison. This shadows specs2's === matcher, causing test assertions to return boolean values instead of proper specs2 Results. Changes: - Split MemoizeSpec into version-specific files - Scala 2.13: Use mustEqual which works with specs2 4.x - Scala 3.3.6: Use must be_== to avoid Box's === method - Changed S.initIfUninitted to S.init with Full(Req.nil) for proper session context initialization 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The Box class defines its own === operator that returns Boolean for value comparison. This shadows specs2's === matcher, causing test assertions to return boolean values instead of proper specs2 Results. Changes: - Split WebSpecSpec into version-specific files - Scala 2.13: Use must_=== which works with specs2 4.x - Scala 3.3.6: Use must be_== to avoid Box's === method - Mark template processing test as pending on Scala 3 due to ClassCastException with snippet instantiation (deeper Scala 3 compatibility issue that needs separate investigation) Test Results: - Scala 2.13: 13 examples, 0 failures ✅ - Scala 3.3.6: 12 passed, 1 pending ✅ 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Split LAFutureWithSessionSpec, MockWebSpec, and LocSpec into Scala 2.13 and 3.3.6 specific versions to handle Box equality matcher differences between specs2 4.x and 5.x. Changes: - Scala 2.13: Uses mustEqual and must_=== for Box/Boolean comparisons - Scala 3: Uses must be_== to avoid Box's === operator - MockWebSpec: Added missing contentType header for JSON test All tests now pass on both Scala 2.13.16 and 3.3.6. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…flection Resolves snippet class instantiation failures in Scala 3.3.6 that were causing ClassCastException when constructing snippet instances via reflection. Root Cause: Scala 3's stricter type inference would default generic type parameter T to Nothing when calling makeOne[T] without explicit type context, causing attempts to cast snippet instances to scala.runtime.Nothing$ which failed. Solution: - Changed ConstructorType.makeOne methods to return AnyRef instead of T - Added targetClass parameter to ConstructorType trait and implementations - Used Class.cast() for runtime type verification before final cast - Moved final asInstanceOf[T] to call site where type context is available - Fixed variable shadowing issue (paramClz vs clz) in constructFrom - Enhanced error handling for ClassCastException and InvocationTargetException The fix uses Class.cast() which performs proper runtime type checking, making the code more type-safe while working correctly in both Scala 2.13 and Scala 3.3.6. Tests Fixed: - OneShot: all 6 tests now pass (snippet callbacks execute correctly) - ToHeadUsages: all 12 tests now pass (snippet rendering works) - WebSpecSpec: template processing test un-pended and passing - Full test suite: 281 tests pass on Scala 3.3.6 (previously had failures) - No regressions: 282 tests pass on Scala 2.13 (unchanged) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
* * Improved Lift session comet handling with additional options for comet creation and initialization functions. * Updated dependencies to newer Jakarta and Apache versions and replaced deprecated methods. * Refactored liftContextRelativePath() to a function returning a List[String] for the Lift-path. This provides more flexibility when interacting with other frameworks. * Removed unused persistence_api dep * Fixed issue caught by copilot * * Keep old signature for findOrCreateComet * Removed Mailer and BCrypt * Reverted changes now that we have the old createComet signature
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 makes Lift 4.0 the Lift that is on
main. Now that it's successfully building for Scala 2 and Scala 3, I think it's time.