Skip to content

WIP: Port Mojolicious 9.49 and stabilize Mojo::IOLoop - #1129

Draft
fglock wants to merge 94 commits into
masterfrom
fix/issue-1115-mojolicious
Draft

WIP: Port Mojolicious 9.49 and stabilize Mojo::IOLoop#1129
fglock wants to merge 94 commits into
masterfrom
fix/issue-1115-mojolicious

Conversation

@fglock

@fglock fglock commented Aug 26, 2026 •
edited
Loading

Copy link
Copy Markdown
Owner

WIP

Implements PerlOnJava support for Mojolicious 9.49 and the runtime/compiler fixes tracked by #1115. No CPAN module sources were patched.

The branch includes permanent project-owned regressions for observed runtime/compiler defects, including the DBIx::Class t/52leaks.t branch regression and inherited Mojolicious listener handling.

UAT example

examples/mojolicious/ adds a loopback live task board with HTML, JSON, and timer-driven chunked output. Its README documents private jcpan -i Mojolicious installation, PERL5LIB fallback, bounded daemon startup, and real-fork limitations. Route-level UAT passes on both JVM and interpreter backends.

UAT follow-up in progress

  • Added project-owned unit/mro_stash_alias_descendants.t, including the core UNIVERSAL::isa(\substr(...), 'LVALUE') behavior; validated on system Perl before implementation and passing on JVM and interpreter backends.
  • Fixed descendant package-stash aliases, aliases assigned through a stash reference, source-stash rebinds, and the LVALUE branch of the unblessed UNIVERSAL::isa matrix.
  • The core alias regressions are now mro/package_aliases.t 54/54, mro/package_aliases_utf8.t 52/52, and mro/isa_aliases.t 13/13. op/universal.t is 134/142. The four-file UAT is 253/261, up from 163/261.
  • Remaining work: eight op/universal.t failures covering Version diagnostics, import/DOES behavior, method lists, and the final @UNIVERSAL::ISA case.
  • The latest isolated full make gate compiled and package-verified, but failed required unit/typeglob.t and unit/regex/re_debug_thread_region.t; investigating before merge.

Validation

  • make: PASS at 1091c8886; subsequent isolated gates compile/package successfully with only the known unrelated unit failure above.
  • Mojolicious 9.49: nice -n 10 timeout 3600 ./jcpan -t Mojolicious — 109 files / 4,194 tests, PASS in 1,186 seconds. Only upstream TEST_* developer or optional-feature paths skip.
  • Catalyst::Runtime: 199 supported files pass; t/live_fork.t remains excluded because real fork is unsupported.
  • DBIx::Class: 325 files / 43,020 tests, PASS, including t/52leaks.t and t/storage/savepoints.t.

The WIP changelog and Mojo::IOLoop module record contain the compact release summary and acceptance evidence. This remains a draft PR for review.

Relates to #1115 and follow-up pseudo-fork proposal #1144.

Fixes #1115.

fglock and others added 30 commits August 25, 2026 14:26
@fglock @codex
test: cover polling listening sockets for read and write
dc706ba 
Add a focused regression for the IO::Poll semantics used by
Mojolicious reactors when a listener is watched for both directions.

Issue: #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: ignore invalid write interest on listening sockets
837b55a 
IO::Poll now limits POLLOUT registration to selectable channels that
support OP_WRITE, while retaining OP_CONNECT for pending client sockets.
This prevents Mojolicious reactors from crashing when they watch a
listener for both readable and writable events.

Issue: #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover overloaded filehandle path coercion
6cc3f34 
Add a focused regression for issue #1115 covering real-file open,
syswrite, selected output, and read behavior when a filename is an
overloaded blessed scalar reference.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: stringify blessed open filenames
efbbb75 
Treat only unblessed scalar references as scalar-backed open targets.
Blessed references remain filename values so overload can produce paths,
preventing Mojolicious log and asset writes from corrupting path scalars.

Fixes #1115.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover module_true do-file return values
e0f5c22 
Lock down Perl 5.38 semantics that module_true does not replace the
value returned by do FILE, including application-like object and false
results.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: preserve do-file values with module_true
0415842 
Apply the module_true true-value substitution only while loading files
through require. Preserve the actual value from do FILE so application
loaders retain blessed objects and false results retain Perl semantics.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover split gzip response inflation
8be337b 
Add a focused Compress::Raw::Zlib regression for the split gzip header
used by Mojolicious response parsing in issue #1115.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover weak state singleton lifecycle
a8d997f 
Add a focused regression for the Mojo::Promise pattern where a weak
attribute points to a destructor-enabled singleton retained by a state
lexical while another temporary alias is released.

Refs #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover malformed loaded-source diagnostics
ad72b5e 
Lock down Perl-compatible missing-delimiter and EOF diagnostics for the
incomplete bareword block used by Mojolicious loader exception tests.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover closure-retained discarded Promise clone
b2298d1 
Reproduce the Mojo::Promise ownership shape where a weak-accessor clone is
returned, discarded by the caller, and retained through a callback closure
stored on the source object. Perl keeps the clone and its weak loop attribute
alive until the callback graph is released.

Refs #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: call imported gzip constants explicitly
cb9091f 
Use callable syntax for imported Compress::Raw::Zlib constants so the
Mojolicious gzip regression compiles under both PerlOnJava backends.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: decode streaming gzip responses
9c23ad6 
Teach the Compress::Raw::Zlib Java backend to consume split RFC 1952
headers, inflate the raw payload, validate the trailer, and report stream
completion so Mojolicious can replace compressed response headers.

Issue: #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: preserve missing-delimiter diagnostics for bareword blocks
f468075 
Detect EOF after parsing an unresolved bareword call's block argument and
route it through the established Perl-compatible missing-right-curly
formatter instead of emitting an empty generic syntax context.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover die after lexical filehandle cleanup
0e3d0c0 
Reproduce the stale input-handle context exposed by Mojolicious template
exceptions after earlier lexical file reads.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover loaded-source EOF line ownership
108c4dd 
Require malformed source loaded from a real temporary file to report its last
content line rather than the synthetic line after a trailing newline.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: preserve loaded-source EOF line ownership
75629e9 
Attribute missing-delimiter diagnostics in newline-terminated loaded files to
the final physical source line while retaining eval and -e next-line behavior.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: make lexical handle regression cwd independent
b37c80e 
Use a temporary input file because the Gradle unit harness supplies a logical
relative __FILE__ name that is not readable from its process directory.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: assert discarded Promise chain destruction
72f0136 
Strengthen the closure-retained Promise regression with deterministic
destruction checks for both an explicitly released source and an entirely
discarded chain at the statement boundary.

Refs #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: preserve Mojo Promise lifecycle roots
a3cf3c2 
Return an empty list from Scalar::Util::weaken in list context, matching its
void XS semantics. This keeps Mojo::Base weak accessors from injecting an
undef argument while cloning Promise objects.

Traverse persistent state slots from installed code roots so weak-reference
sweeps retain singleton objects such as Mojo::IOLoop.

Refs #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: guard returned lexical filehandle lifetime
73351f0 
Verify that subroutine-exit diagnostic cleanup does not close or unregister a
lexical filehandle returned to its caller.

Generated with [OpenAI Codex](https://openai.com/codex)

Co-Authored-By: OpenAI Codex <codex@openai.com>
@fglock @codex
test: cover Promise warning cleanup timing
0d5ed75 
Model rejected Promise propagation through a next-tick callback queue and
require discarded-chain destruction while the warning handler is localized.

Refs #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: clear stale lexical readline diagnostics
cb553d7 
Clear the JVM's last-read handle when an unaliased lexical IO owner leaves via
the subroutine return cleanup path. Preserve holder counts, descriptors, and
returned or copied handle aliases.

Generated with [OpenAI Codex](https://openai.com/codex)

Co-Authored-By: OpenAI Codex <codex@openai.com>
@fglock @codex
test: mirror Mojo Promise wait cleanup
a356ea7 
Strengthen the warning cleanup regression with Mojo's wait callback shape so
the interpreter-specific destruction timing failure is isolated.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: mirror paired Promise callback cleanup
7ad8fb2 
Model Mojo::Promise's paired resolve and reject callback arrays so the
interpreter must release the unselected branch before warning scope exit.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: classify runtimes without real fork support
b9ff3a6 
Require Config to advertise either real fork or the standard non-real fork
classification so upstream suites can avoid process-only tests.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover tempfile missing-parent diagnostics
f30831b 
Require File::Temp to identify a nonexistent parent directory before retrying
temporary-file creation.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: classify PerlOnJava fork as non-real
0e106ff 
Advertise the non-real fork classification while keeping d_fork disabled so
portable upstream suites skip process-only semantics.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: report missing File::Temp parent directories
8fddc42 
Validate the template parent before retrying exclusive creation and preserve
the standard diagnostic distinction between missing and non-directory paths.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: mirror Mojo Promise finally cleanup
23ebcd2 
Model Mojo::Promise settlement, finally chaining, and wait cleanup closely
enough to expose the interpreter's delayed unhandled-rejection warning.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover File::Path option semantics
1fbf63f 
Require make_path to initialize its error collector and remove_tree to honor
keep_root while deleting descendants.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
fglock and others added 30 commits August 26, 2026 00:15
@fglock @codex
test: cover explicitly returned socket lifetime
331d465 
Verify that an explicit return keeps a socket open for the caller and that
releasing the returned value then delivers EOF to its peer. This reproduces
an ownership-transfer leak on both PerlOnJava backends.

Refs #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover aliases created before socket initialization
533d504 
Reproduce phantom IO holder counts when a gensym is aliased before socketpair
attaches an IO slot and the constructed handle is returned to its caller.

Refs #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover socket ownership in returned lists
4d630b1 
Reproduce the IO::Socket::IP accept pattern where a socket returned in a
temporary list is assigned to a caller lexical. The temporary list must not
retain a phantom descriptor owner after the caller releases the handle.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: preserve caller ownership across socket methods
763216b 
Cover method-style argument handling where a shifted socket alias is inspected
inside a call. The caller must retain ownership after the method lexical exits.

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: close stream sockets at final Perl owner
201f0f2 
Track anonymous socket ownership across lexical scope cleanup, containers,
closure captures, argument aliases, and explicit returns on both execution
backends. Transfer ownership from list-assignment temporaries without stealing
caller ownership from method arguments, so IO::Socket::IP accept handles close
at the correct point and Mojolicious observes EOF.

Update the IO lifecycle design document with the completed ownership phase and
remaining non-socket limitations.

Fixes #1115

Generated with [Codex](https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover repeated Mojolicious mount requests
59618fc 
Add a self-contained optional Mojolicious regression for the mounted app
lifecycle failure from issue #1115. The test passes on system Perl and
fails on the unfixed JVM backend after the first mounted request tears
down the external renderer namespace.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: preserve live mounted routes during request cleanup
a91f766 
Keep weakly observed blessed objects alive when Perl-visible root walkers
still prove ownership at a transient zero-count boundary. This prevents a
request-local Mojolicious route match from recursively destroying a mounted
application and its generated renderer helper namespace.

Document the lifecycle rule and make the focused Mojolicious regression
portable to the embedded unit-test harness.

Fixes #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover skipped callback closure cleanup
eb6f185 
Preserve a focused regression for issue #1115 where cleanup of a skipped
Test::More callback destroys state captured from a still-live outer scope.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: preserve captured lexicals across skip exits
112da7a 
Exclude constructor-captured closure slots from non-local loop-control scope
cleanup so Test::More skip callbacks cannot mark enclosing lexicals exited.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover gzip and zlib inflate auto-detection
3b10373 
Add a system-Perl-validated regression for WANT_GZIP_OR_ZLIB input.
The unfixed JVM rejects both supported wrapper formats.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: auto-detect gzip and zlib inflate wrappers
f34c936 
Implement the MAX_WBITS + 32 mode used by WANT_GZIP_OR_ZLIB in the
bundled Compress::Raw::Zlib Java bridge. Preserve streaming header
detection and select the appropriate gzip or zlib inflater.

This fixes Catalyst Runtime's compressed UTF response charset path.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover shifted weak reference to deep cycle
4d5da70 
Model the multi-container strong cycle diagnosed by DBIx::Class t/52leaks.t
and require an inline shifted weak diagnostic to remain live until the cycle
is explicitly broken.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover recursive DBIC weak registry diagnostics
dc9db6d 
Model a pending DESTROY rescue while DBIx-style leak tracing checks an
unrelated strong cycle through Scalar::Util::isweak.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: keep cycle weak checks observational
c6f43fd 
Require DBIx-style isweak diagnostics on a strong-cycle member to leave
pending DESTROY rescues and the inspected cycle untouched.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: mirror DBIC inner leak registry call
712bf08 
Name the focused helper and its one-entry registry after DBIx::Class's
actual inner assert_empty_weakregistry diagnostic path.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix: preserve DBIC inner leak diagnostics
f66ab34 
Protect strong cycle islands during targeted release sweeps and limit the
DBIC rescued-object compatibility sweep to its outer leak registry. Inspect
the active assert_empty_weakregistry argument so the one-entry inner cycle
diagnostic remains observational without patching DBIx::Class.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
docs: summarize Mojolicious support
7e67612 
Add the issue #1115 web-stack and runtime work to the compact work-in-progress
changelog.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
test: cover DBIx quiet leak-registry handoff
3d9e2ec 
Model the DBIx::Class schema destructor handoff that must be collected before a
quiet one-entry weak registry is checked.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix(runtime): clean quiet DBIx leak registries
54fc20f 
Treat quiet LeakTracer END checks as outer cleanup points even when their weak
registry has only one entry. Remove the JVM-specific call-depth restriction so
the same narrow package, subroutine, and argument checks work in the interpreter.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
docs: record final issue 1115 acceptance
ebdfeff 
Mark the DBIx regression follow-up complete and record the exact full-suite
acceptance totals.

Refs #1115

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
fix(io): preserve inherited listener blocking mode
8032d14 
Follow transparent IO wrappers when changing blocking mode so Mojo servers
opened with new_from_fd do not block after accepting their first connection.
Add focused Mojolicious coverage and record final issue 1115 acceptance.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
docs: record Mojolicious acceptance follow-up
edf3a13 
Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock @codex
docs(examples): add Mojolicious task board UAT
1091c88 
Add a loopback task-board example with HTML, JSON, and timer-streaming routes,
plus private jcpan installation and single-process deployment guidance.

Generated with [Codex](https://openai.com/codex/)

Co-Authored-By: Codex <codex@openai.com>
@fglock
fix: preserve stash aliases across source rebinds
00766c8 
Keep descendant package identities when a stash source is rebound, and cover
the mro namespace-move regression from Perl core package_aliases.t.

Generated with Codex (https://openai.com/codex/)

Co-Authored-By: Codex <223556219+chatgpt-codex-connector[bot]@users.noreply.github.com>
@fglock
fix: retain namespace metadata for moved nested stashes
65e7c4d 
Recreate every moved RuntimeStash view at its destination so namespace deletion
targets the surviving alias path after a source stash is rebound.

Generated with Codex (https://openai.com/codex/)

Co-Authored-By: Codex <223556219+chatgpt-codex-connector[bot]@users.noreply.github.com>
@fglock
fix: resolve aliases for packages ending in colon
6d0fea4 
Treat three-colon glob spellings as package stashes and canonicalize main:::
to the root : package so MRO aliases resolve consistently.

Generated with Codex (https://openai.com/codex/)

Co-Authored-By: Codex <223556219+chatgpt-codex-connector[bot]@users.noreply.github.com>
@fglock
fix: delete colon-ended package namespaces
64165db 
Route colon-ended parent-stash keys through namespace deletion so stale stash
aliases cannot remain visible to MRO after delete.

Generated with Codex (https://openai.com/codex/)

Co-Authored-By: Codex <223556219+chatgpt-codex-connector[bot]@users.noreply.github.com>
@fglock
fix: treat shared stash aliases as isa-equivalent classes
37974f0 
Allow empty shared stashes through UNIVERSAL::isa and compare canonical stash
identities across the linearized hierarchy.

Generated with Codex (https://openai.com/codex/)

Co-Authored-By: Codex <223556219+chatgpt-codex-connector[bot]@users.noreply.github.com>
@fglock
fix: recognize substr lvalues in UNIVERSAL isa
884a012 
Match Perl's unblessed-reference matrix so references to substr lvalues are
LVALUE, not SCALAR. Extend the issue #1115 stash alias regression test with
the observed core-test behavior.

Generated with Codex (https://openai.com/codex/)
Co-Authored-By: Codex <223556219+chatgpt-codex-connector[bot]@users.noreply.github.com>
@fglock
fix: resolve IO invocants in UNIVERSAL can
3cc7e8c 
Treat bare globs and handle barewords with a live IO slot as IO::Handle
invocants before generic package-name resolution. Add a system-Perl-validated
regression for all bare glob, glob-reference, and bareword forms.

Generated with Codex (https://openai.com/codex/)
Co-Authored-By: Codex <223556219+chatgpt-codex-connector[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Port Mojolicious 9.49 and stabilize Mojo::IOLoop

1 participant

@fglock