My 78 open PRs, grouped

[u9g]

PRs waiting on a minecraft-data PR

minecraft-data PR	mineflayer PR	mineflayer PR state
PrismarineJS/minecraft-data#1273 (open)	#4062	not ready, green — approved; waits on the minecraft-data release
PrismarineJS/minecraft-data#1275 (merged 7 Sep, not in a release)	#4063	not ready, green — safe to merge, sneak fix inert until the minecraft-data release
PrismarineJS/minecraft-data#1278 (open)	#4089	not ready, red — waits on the minecraft-data release
PrismarineJS/minecraft-data#1282 (open)	#4089	not ready, red — conflicting, and waits on the minecraft-data release

PRs waiting on a protodef release

Prereq	mineflayer PR	State
PrismarineJS/prismarine-item#184 merged and released	#4027	not ready, red — waits on PrismarineJS/prismarine-item#184
ProtoDef-io/node-protodef#177 and ProtoDef-io/ProtoDef#65 merged and released	PrismarineJS/prismarine-item#184	not ready, red — waits on the protodef release; design question open
nothing; awaiting re-review	ProtoDef-io/node-protodef#177, ProtoDef-io/ProtoDef#65	ready, green — pushed a response to @extremeheat's 6 Sep review on 12 Sep; the five threads are still open

PRs that fix minecraft-data's data

Wire-layout and feature-flag fixes, each checked against a decompiled STREAM_CODEC. The recipe pair is unrelated to the rest and closes PrismarineJS/minecraft-data#182.

PR	Status
PrismarineJS/minecraft-data#1267	ready, green
PrismarineJS/minecraft-data#1273	ready, green — duplicates PrismarineJS/minecraft-data#1154 and PrismarineJS/minecraft-data#1251
PrismarineJS/minecraft-data#1278	not ready, red — protocol.json desynced from the yaml, needs npm run build
PrismarineJS/minecraft-data#1282	ready, green
PrismarineJS/minecraft-data#1284	not ready, green — @extremeheat's question unanswered; design question open
PrismarineJS/minecraft-data#1292	ready, green — @extremeheat said on 12 Sep he might fix this himself
PrismarineJS/minecraft-data#1283	not ready, green — after PrismarineJS/minecraft-data-generator#81
PrismarineJS/minecraft-data-generator#81	ready, green — produces PrismarineJS/minecraft-data#1283's data

PRs ready to review

Green CI, mergeable, not waiting on another PR or a release.

PR	Status
#4021	ready, green — was your only objection
#4083	ready, green — closes #1587, half of #3274
#4064	ready, green
#4067	ready, green — also fixes one of the master CI flakes
#4068	ready, green
#4069	ready, green
#4071	ready, green
#4073	ready, green
#4085	ready, green
#4086	ready, green
#4087	ready, green
#4088	ready, green
#4093	ready, green
#4100	ready, green
#4101	ready, green
#4102	ready, green
#4103	ready, green
#4104	ready, green
#4107	ready, green
#4108	ready, green
#4111	ready, green — same file as #4112, second one rebases
#4112	ready, green — same file as #4111, second one rebases
#4115	ready, green
#4116	ready, green
#4118	ready, green
PrismarineJS/mineflayer-pathfinder#375	ready, green
PrismarineJS/mineflayer-pathfinder#376	ready, green
PrismarineJS/mineflayer-pathfinder#377	ready, green
PrismarineJS/mineflayer-pathfinder#378	ready, green
PrismarineJS/mineflayer-pathfinder#379	ready, green
PrismarineJS/mineflayer-pathfinder#380	ready, green
PrismarineJS/mineflayer-pathfinder#381	ready, green
PrismarineJS/node-minecraft-protocol#1522	ready, green
PrismarineJS/node-minecraft-protocol#1523	ready, green
PrismarineJS/node-minecraft-protocol#1524	ready, green
PrismarineJS/node-minecraft-protocol#1527	ready, green — unblocks PrismarineJS/node-minecraft-protocol#1526
PrismarineJS/prismarine-item#185	ready, green
PrismarineJS/prismarine-item#188	ready, green — read after PrismarineJS/prismarine-item#189
PrismarineJS/prismarine-item#189	ready, green
ProtoDef-io/node-protodef#178	ready, green
ProtoDef-io/node-protodef#179	ready, green
ProtoDef-io/node-protodef#180	ready, green
ProtoDef-io/node-protodef#177	ready, green — pushed a response to @extremeheat's review on 12 Sep, five threads still open
ProtoDef-io/ProtoDef#65	ready, green — moves with ProtoDef-io/node-protodef#177
PrismarineJS/minecraft-data#1292	ready, green — @extremeheat said on 12 Sep he might fix this himself
PrismarineJS/prismarine-viewer#488	ready, green
PrismarineJS/prismarine-viewer#490	ready, green — closes PrismarineJS/prismarine-viewer#57
PrismarineJS/prismarine-viewer#506	ready, green
PrismarineJS/prismarine-viewer#509	ready, green
PrismarineJS/prismarine-viewer#513	ready, green
PrismarineJS/prismarine-viewer#514	ready, green

(51 PRs)

PRs blocked on me

PR	State	Why
#4065	not ready, green	@rom1504 asked on 7 Sep why this is needed and said it does not follow the plugin interface; no reply from me, thread still open
PrismarineJS/minecraft-data#1284	not ready, green	@extremeheat asked on 9 Sep whether the field is a fixed enum or variable, and expanded on 11 Sep after my last push; no reply from me
PrismarineJS/minecraft-data#1278	not ready, red	CI says 26.1 (26.1) / protocol.json is desynced from yaml, please run 'npm run build'; the PR edits protocol.json without regenerating
PrismarineJS/prismarine-viewer#503	not ready, green	based on a branch that merged as PrismarineJS/prismarine-viewer#501; needs retargeting to master
PrismarineJS/prismarine-viewer#484	not ready, red	build (1.14.4) renders an empty frame; the dep releases it was waiting on have all shipped

Other states

Red CI from master, not from the PR

Repo and job	Failure	Affected PRs
mineflayer — MC 1.16.5 1.17.1 1.18.2 1.19 red since 7 Sep, MC 1.19.2 1.19.3 1.19.4 1.20.1 since 12 Sep	One flaky external test per run, a different one each time (master run)	#4074, #4075, #4081, #4099
mineflayer-pathfinder — build (24.x)	Physics test: Simulated states don't match Bot: (0.5, 3.7967356006686925, 13.30954454956581) !== Simulation: (0.5, 3.495200877005912, 13.45929100856761)	PrismarineJS/mineflayer-pathfinder#382, on the identical assertion

Stacked on another open PR's branch

PR	State	Based on
#4094	not ready, green	#4085
PrismarineJS/prismarine-viewer#510	not ready, green	PrismarineJS/prismarine-viewer#490
PrismarineJS/prismarine-viewer#511	not ready, green	PrismarineJS/prismarine-viewer#510

Ordering, both sides on master

PR	State	Order	Reason
PrismarineJS/node-minecraft-protocol#1526	not ready, red	after PrismarineJS/node-minecraft-protocol#1527	PrismarineJS/node-minecraft-protocol#1526 un-hides the suite PrismarineJS/node-minecraft-protocol#1527 fixes, so all its jobs are red until then
PrismarineJS/node-minecraft-protocol#1525	ready, green	either	its new tests only start running in CI once PrismarineJS/node-minecraft-protocol#1526 and PrismarineJS/node-minecraft-protocol#1527 land
#4114	ready, green	either, #4085 preferred first	#4085 replaces the sequence: 0 this writes with a real shared counter
#4111, #4112	ready, green	either	both touch lib/scoreboard.js; whichever lands second needs a rebase
PrismarineJS/minecraft-data#1283	not ready, green	after PrismarineJS/minecraft-data-generator#81	PrismarineJS/minecraft-data-generator#81 produces the data PrismarineJS/minecraft-data#1283 commits
PrismarineJS/prismarine-physics#142	not ready, green	after #4083	#4083 exposes the flight state PrismarineJS/prismarine-physics#142 acts on

Open design questions

PR	State	Question
PrismarineJS/minecraft-data#1284	not ready, green	It replaces a numeric enum mapper with a plain varint. @extremeheat's position is the opposite: eliminate the numeric enum datatypes across the protocol data. I have no preference. PrismarineJS/minecraft-data#1278 makes the same change to use_entity.hand, so whatever is decided here applies to it, but nobody has commented there. PrismarineJS/minecraft-data#1292 reverts four other 1.21.6 mappers on the same reasoning and keeps entity_action.actionId as a mapper, which is the case for naming them.
PrismarineJS/prismarine-item#184	not ready, red	Whether HashedSlot hashing belongs in a generic hash protodef type (ProtoDef-io/node-protodef#177, ProtoDef-io/ProtoDef#65) or in prismarine-item. @extremeheat's feedback on where the hashing code should live is on ProtoDef-io/node-protodef#177; I pushed a response there on 12 Sep.

Duplicates

PR	State	Duplicates
PrismarineJS/minecraft-data#1273	ready, green	PrismarineJS/minecraft-data#1154 and PrismarineJS/minecraft-data#1251, both open from other contributors

---

A. Vanilla-client packet parity — 18

What mineflayer puts on the wire, diffed against the decompiled client (LocalPlayer.tick, MultiPlayerGameMode, ServerGamePacketListenerImpl). The symptom is usually that something works against a permissive server and does nothing against one that checks. Split out of #4066.

PR	Title	Status
#4063	physics: send the movement input packets the vanilla client sends	not ready, green — safe to merge, sneak fix inert until the minecraft-data release
#4064	health: send player_loaded when the vanilla client would	ready, green
#4065	Send the client brand and Client Information when the vanilla client does	not ready, green — @rom1504's question unanswered
#4069	send held_item_slot only when the slot differs from the slot the server holds	ready, green
#4085	use_item and use_item_on like the vanilla client: shared prediction sequence, cursor on the clicked face, no empty-hand use, swing after the packet	ready, green
#4094	Swing every tick while digging, skip STOP on instant breaks and abort with face DOWN	not ready, green — stacked on #4085
#4086	Send the client_command respawn after win_game with the actionId field	ready, green
#4087	Report resource packs as ACCEPTED, DOWNLOADED, SUCCESSFULLY_LOADED and deny once	ready, green
#4089	Right-click entities like the vanilla client: interact pair, real sneak state, hit point	not ready, red — waits on the minecraft-data release
#4093	Dismount by holding sneak for one tick on 1.21.3+	ready, green
#4107	Answer pings and teleports at the start of the next tick, in arrival order	ready, green
#4108	physics: a teleport leaves the standing flag alone	ready, green
#4114	placeEntity: write a complete use_item when placing a boat	ready, green — #4085 first is preferred
#4115	placeBlock: derive the face and the cursor from a raycast, like the client does	ready, green
#4118	Expose the interaction ranges the server sends, and the reach check vanilla makes	ready, green
PrismarineJS/node-minecraft-protocol#1522	Match the vanilla client in the login/configuration handshake: cookies, brand, per-login chat session, known packs	ready, green
PrismarineJS/node-minecraft-protocol#1523	write the 0 holder id for inline registryEntryHolder entries	ready, green
PrismarineJS/node-minecraft-protocol#1525	send a command with no signable argument as chat_command	ready, green — after PrismarineJS/node-minecraft-protocol#1526

B. mineflayer state tracking and API surface — 18

Not packet parity: what mineflayer remembers. State surviving a login it should not, an event fired without its argument, a getter returning the wrong shape.

PR	Title	Status
#4021	placeBlock: reject an occupied target up front	ready, green — was your only objection
#4027	inventory: claim clicked slots by hash on 1.21.5+	not ready, red — waits on PrismarineJS/prismarine-item#184
#4062	entities: read the 1.21.2+ entity_teleport layout	not ready, green — approved; waits on the minecraft-data release
#4068	drop the physics tick backlog after an event-loop stall instead of draining it	ready, green
#4071	reset teams and scoreboards on every login	ready, green
#4073	make bot.controlState's properties enumerable	ready, green
#4081	give teamRemoved the team that was removed	not ready, red — master CI is red, not this PR
#4083	Track the abilities the server grants	ready, green — closes #1587, half of #3274
#4099	cancel the deferred respawn reply once it is stale, and write no movement packets outside play	not ready, red — master CI is red, not this PR
#4100	Make the named scoreboard display slots non-enumerable	ready, green
#4101	Throw a descriptive error when bot.chat is called before login	ready, green
#4102	Normalize open_window titles to ChatMessage	ready, green
#4103	Window state across close, re-login and respawn: stateId per window, close_window on respawn, closeWindow(null)	ready, green
#4104	Accept play-phase resource packs when no resourcePack listener answers them	ready, green
#4111	Read the scoreboard objective title as a chat component	ready, green — same file as #4112, second one rebases
#4112	scoreboard: read the 1.20.3+ objective and score packets	ready, green — same file as #4111, second one rebases
#4116	waitForTicks: say when it is physics that is missing, not the server	ready, green
PrismarineJS/prismarine-physics#142	Obey the flight the server grants in the abilities packet	not ready, green — after #4083, which exposes the state it acts on

C. prismarine-item: 1.20.5+ item components — 4

Components are on the wire but items are still read as pre-1.20.5 NBT, so name, lore and custom data come back in the wrong shape or not at all.

PR	Title	Status
PrismarineJS/prismarine-item#184	Item.toHashedNotch for 1.21.5+ window_click slot claims	not ready, red — waits on the protodef release; design question open
PrismarineJS/prismarine-item#185	Expose the custom_data component as item.nbt on 1.20.5+	ready, green
PrismarineJS/prismarine-item#188	Render an item's custom name in displayName	ready, green — read after PrismarineJS/prismarine-item#189
PrismarineJS/prismarine-item#189	Read the custom name and lore components as the JSON string they used to be	ready, green

D. prismarine-viewer — 10

Two threads: the viewer drawing the wrong thing (nothing below y=0, no skins, no dropped items, invisible entities drawn), and a host/mesher split so the render path stops holding the event loop (PrismarineJS/prismarine-viewer#490 → PrismarineJS/prismarine-viewer#510 → PrismarineJS/prismarine-viewer#511).

PR	Title	Status
PrismarineJS/prismarine-viewer#484	Add Minecraft 26.1 support	not ready, red — empty render on 1.14.4
PrismarineJS/prismarine-viewer#488	Render negative-Y sections on 1.18+ worlds	ready, green
PrismarineJS/prismarine-viewer#490	Render player skins	ready, green — closes PrismarineJS/prismarine-viewer#57
PrismarineJS/prismarine-viewer#506	Render dropped items	ready, green
PrismarineJS/prismarine-viewer#509	resolve entity bone parents by name, whatever their case	ready, green
PrismarineJS/prismarine-viewer#513	Unload chunks the server unloads, and follow the world across a transfer	ready, green
PrismarineJS/prismarine-viewer#514	Draw nothing for an invisible entity, like the vanilla client	ready, green
PrismarineJS/prismarine-viewer#503	viewer.waitForReady() for a fully textured first frame	not ready, green — conflicting, needs retargeting to master
PrismarineJS/prismarine-viewer#510	Decode host images without holding the event loop	not ready, green — stacked on PrismarineJS/prismarine-viewer#490
PrismarineJS/prismarine-viewer#511	Parse the block states in the mesher, not on the calling thread	not ready, green — stacked on PrismarineJS/prismarine-viewer#510

E. mineflayer-pathfinder createHuman — 8

createHuman behaving unlike a player. Independent of each other and of the groups above.

PR	Title	Status
PrismarineJS/mineflayer-pathfinder#375	goto rejects an unreachable goal instead of resolving; stop() no longer latches while idle	ready, green
PrismarineJS/mineflayer-pathfinder#376	cap the wait for the head to settle	ready, green
PrismarineJS/mineflayer-pathfinder#377	a walkTo for the goal in flight joins that walk instead of superseding it	ready, green
PrismarineJS/mineflayer-pathfinder#378	take three-block drops	ready, green
PrismarineJS/mineflayer-pathfinder#379	hold the controller through a setback storm	ready, green
PrismarineJS/mineflayer-pathfinder#380	a walk that stops under an unreachable goal rejects instead of resolving	ready, green
PrismarineJS/mineflayer-pathfinder#381	walkTo and lookAt say so when physics is disabled	ready, green
PrismarineJS/mineflayer-pathfinder#382	bridge over gaps the way a player does	not ready, red

F. Test and CI plumbing — 7

No behaviour change. Tests passing for the wrong reason, or suites that have never run. The old -g <version> filter in nmp meant declare_commands handling, lpVec3 and ping have never run in any job.

PR	Title	Status
#4067	test(nether): let retries pass by respecting the player's portal cooldown	ready, green — also fixes one of the master CI flakes
#4074	test: gate the no-chunk physics check on a chunk load instead of a 2 s sleep	not ready, red — master CI is red, not this PR
#4075	test: build only the versions under test	not ready, red — master CI is red, not this PR
#4088	internal tests: wait for plugin injection before driving the bot from playerJoin	ready, green
PrismarineJS/node-minecraft-protocol#1524	test: retry the vanilla ping while the server has no status yet	ready, green
PrismarineJS/node-minecraft-protocol#1526	test: select the version under test with MC_VERSION instead of a title grep	not ready, red — red until PrismarineJS/node-minecraft-protocol#1527 lands
PrismarineJS/node-minecraft-protocol#1527	test: give the declare_commands tree the packet's rootIndex	ready, green — unblocks PrismarineJS/node-minecraft-protocol#1526

G. ProtoDef, upstream — 5

Not PrismarineJS, but the same reviewers and PrismarineJS work sits behind them. ProtoDef-io/node-protodef#177 and ProtoDef-io/ProtoDef#65 are the hash datatype for PrismarineJS/prismarine-item#184; the rest came from chasing unreadable packets in the harness.

PR	Title	Status
ProtoDef-io/node-protodef#177	Add the hash datatype	ready, green — pushed a response to @extremeheat's review on 12 Sep, five threads still open
ProtoDef-io/ProtoDef#65	Add the hash datatype — spec side of ProtoDef-io/node-protodef#177	ready, green — moves with ProtoDef-io/node-protodef#177
ProtoDef-io/node-protodef#178	FullPacketParser: emit partialReadError with the chunk it could not read	ready, green
ProtoDef-io/node-protodef#179	Throttle repeated packet-parse error logs in FullPacketParser	ready, green
ProtoDef-io/node-protodef#180	Serializer: name the packet in write errors from a compiled protocol	ready, green

[rom1504]

Astra agent comment — AI-generated, not manually written by the maintainer.

At the maintainer's request, I wrote up the cross-PR refactoring proposals in #4135:

• Shared Mineflayer interaction sending, prediction sequencing and coherent hit results, starting with use_item and use_item_on like the vanilla client: shared prediction sequence, cursor on the clicked face, no empty-hand use, swing after the packet #4085 / placeEntity: write a complete use_item when placing a boat #4114.
• A client-tick coordinator that separates reply scheduling and input synchronization from physics simulation.
• Explicit world/object ownership for viewer asynchronous work, dimension bounds and readiness.

The issue maps the affected PR families, assigns responsibilities to the existing PrismarineJS packages, and gives small first implementation slices and validation steps. It also proposes landing #4126's test split to reduce conflicts. These are proposals for discussion, not a requirement to hold every bounded fix until all refactoring is complete.

Skills used: prismarine-architecture-review guided package ownership and incremental boundaries; the lifecycle, geometry, item/inventory and world/render review skills helped connect the proposals to concrete problems already found in these PRs.

[rom1504]

Astra agent review — AI-generated, not manually written by the maintainer.

Updated-PR review summary — 2026-09-19

At the maintainer's request, three subagents used the PrismarineJS review skills to review the 33 open U9G PRs whose heads changed since the September 18 baseline. This is a dated review snapshot of that subset, not a status update for every PR in this issue.

17 ready · 11 conditional · 5 blocked.

Each PR below links directly to its published review, with the checks run, evidence limits, skills used and remaining conditions. Ready means it looks ready from that review; conditional means a dependency, branch, CI or existing discussion still needs follow-up; blocked means a reproduced code defect remains. These are review assessments, not formal GitHub approvals.

PR / review	Reviewed head	Assessment	Result or remaining action
minecraft-data #1278	237e8a8	Ready	All five hand fields preserve named/numeric compatibility through the production codec.
minecraft-data #1283	633c362	Ready	Recipe data matches generator #81, including cake remainders. Coordinate landing with #81.
minecraft-data-generator #81	b49eacf	Ready	Generator artifacts match data #1283. Land together so regeneration preserves the fix.
mineflayer #4085	ec0d114	Ready	Shared sequence ownership and boat interleaving verified.
mineflayer #4104	8ccaa36	Ready	Resource-pack once-listener ownership verified.
mineflayer #4107	875ff4f	Ready	Teleport/ping ordering and lifecycle queue cleanup verified.
mineflayer #4108	704a251	Ready	Standing state and teleport-reply flags verified.
mineflayer #4112	c3fcdfd	Ready	Score removal semantics, declarations and documentation verified.
mineflayer-pathfinder #379	0e2bf3d	Ready	Setback ownership across awaited work verified.
node-minecraft-protocol #1525	eb2e358	Ready	Acknowledgement handling verified; actual merge result includes the already-merged command fixture fix.
prismarine-item #189	4a14d0a	Ready	Literal component names/lore and legacy behavior verified.
prismarine-physics #142	bb4b53b	Ready	Flight simulator regressions pass; Mineflayer abilities integration is separate.
prismarine-viewer #484	6d08538	Ready	Ready for 26.1 support. Preserve #488's custom-column bounds when combining.
prismarine-viewer #488	4e8666f	Ready	Per-column bounds and below-floor culling verified; preserve when combining with #484.
prismarine-viewer #506	6f1de51	Ready	Dropped-item identity, fallback and resource cleanup verified.
prismarine-viewer #513	3a6c28e	Ready	Stale column completions rejected across unload, replacement and transfer.
prismarine-viewer #514	9413e5b	Ready	Invisibility persists across partial entity updates.
mineflayer #4027	4bac54b	Conditional	Release item #184, raise the minimum dependency, and validate the hashed path.
mineflayer #4062	8b4440b	Conditional	Release the required minecraft-data feature/schema changes.
mineflayer #4093	b8466c6	Conditional	Land and integration-check #4063, including its mounted-input repair.
mineflayer #4094	2c53f8a	Conditional	Refresh the #4085 stack and rerun failed CI (nether spawn timeout).
mineflayer #4115	10cd5d2	Conditional	Geometry fixes pass; rerun failed CI (server bind and scoreboard failures).
mineflayer #4118	0ff8957	Conditional	Range tests pass; obtain normal test CI and settle the existing entity-ownership discussion. Full extraction need not precede this fix.
mineflayer #4127	8c42f90	Conditional	Use a released dependency containing the cursor-schema correction and rerun failed CI.
mineflayer-pathfinder #376	e5fbd00	Conditional	Timeout regressions pass; obtain normal test CI (current checks only cover Socket Security).
node-minecraft-protocol #1522	96c837c	Conditional	Coordinate the documented 1.21.8 stored-cookie schema fix/release.
prismarine-item #184	9714597	Conditional	Release ProtoDef #177 / protodef ^1.20.0 and rerun CI; 197 tests pass against the candidate dependency.
prismarine-viewer #503	626ec89	Conditional	Retarget/rebase from player-skins onto master and recheck CI.
mineflayer #4063	b296ac5	Blocked	Mounted input is suppressed by the physics guard; new inline finding.
mineflayer #4064	0e51bd8	Blocked	Existing healthy-respawn finding reproduced: player_loaded can be sent before the column is ready.
mineflayer #4065	f15a530	Blocked	Existing supplied-client finding reproduced: settings getter is attached to the wrong options object.
mineflayer-pathfinder #382	7f4431d	Blocked	Facing timeout is ignored and the walk reports success; new inline finding. CI also needs follow-up.
prismarine-item #188	3945dc4	Blocked	Existing pre-1.13 plain-name parsing defect reproduced; JSON-looking literal names can be misread or throw.

New inline findings

• Mineflayer #4063: input synchronization stops while mounted. Independently reproduced against base/head physics plugins with compiled packet codecs.
• Pathfinder #382: facing timeout reports successful completion. Independently reproduced with the production controller when physics ticks stop.

The existing blockers on Mineflayer #4064/#4065 and item #188 were independently reproduced and linked in their reviews without opening duplicate inline threads.

The skills helped separate packet/schema and release readiness from model-only checks, trace action ownership across waits and lifecycle changes, verify geometry and renderer state against their actual owners, and keep bounded fixes separate from broader architecture proposals. Each individual review explains the specific skill contributions. Validation details distinguish local protocol fixtures, candidate dependencies and controlled renderer tests from live vanilla-server or browser coverage.

All 33 published reviews were checked against their reviewed commits, attribution and inline locations. Later commits or merges may supersede this snapshot.

[rom1504]

I'm manually reviewing the ready ones

[rom1504]

4107 last reviewed

[rom1504]

Astra agent review — AI-generated, not manually written by the maintainer.

U9G PR status update — 2026-09-21

Checked 2026-09-21 05:00 UTC. This updates the September 19 table using current PR states, heads, checks, conflicts and subsequent discussions, including rom1504’s manual reviews. This is a status refresh, not another code-review/test pass; author-reported fixes are identified as such.

Of the previous 33 PRs: 8 merged and 25 remain open. The 25 open PRs comprise 8 with maintainer authorization to merge after conflict resolution, 3 updated fixes awaiting re-review, 1 new maintainer test request, 9 dependency/CI/discussion follow-ups and 4 unchanged code blockers.

Across PrismarineJS and ProtoDef-io there are 44 open U9G PRs, down from 52 at the previous snapshot, with no newly opened PRs in this set. The additional 19 open PRs are listed separately below.

Landing detail: viewer #503 merged into player-skins. Its merge commit is not contained in master, and master still has no Viewer.waitForReady; that merge therefore still needs integration into master. The other seven merges below targeted their default branches.

PR	Current assessment	Checks	Conflicts	Latest result / next step
minecraft-data #1278	Merged	—	—	Merged 2026-09-19 into master. Discussion/review
minecraft-data #1283	Merged	—	—	Merged 2026-09-19 into master. Discussion/review
minecraft-data-generator #81	Merged	—	—	Merged 2026-09-19 into main. Discussion/review
mineflayer-pathfinder #379	Merged	—	—	Merged 2026-09-21 into master. Discussion/review
prismarine-item #189	Merged	—	—	Merged 2026-09-21 into master. Discussion/review
prismarine-physics #142	Merged	—	—	Merged 2026-09-21 into master. Discussion/review
prismarine-viewer #484	Merged	—	—	Merged 2026-09-21 into master. Discussion/review
prismarine-viewer #503	Merged	—	—	Merged 2026-09-21 into player-skins. Follow-up: this is player-skins, not master; master still lacks waitForReady. Bring the change to master. Discussion/review
mineflayer #4107	Maintainer authorized	Green	No	rom1504: resolve conflicts, then merge. Head rewritten to e09ad6b; GitHub now reports no conflicts and green checks. This update does not independently re-review the rewritten head. Discussion/review
mineflayer #4108	Maintainer authorized	Socket only	Yes	rom1504 approved resolving conflicts and merging; conflicts remain. Discussion/review
mineflayer #4112	Maintainer authorized	Socket only	Yes	rom1504 approved resolving conflicts and merging; conflicts remain. Discussion/review
mineflayer-pathfinder #376	Maintainer authorized	Socket only	Yes	rom1504: resolve conflicts and merge. Conflicts remain; the green rollup only covers Socket checks. Discussion/review
prismarine-viewer #488	Maintainer authorized	Green	Yes	rom1504 approved resolving conflicts and merging; conflicts remain after #484 landed. Discussion/review
prismarine-viewer #506	Maintainer authorized	Green	No	rom1504 approved resolving conflicts and merging. GitHub currently reports no conflicts. Discussion/review
prismarine-viewer #513	Maintainer authorized	Green	No	rom1504 approved resolving conflicts and merging. GitHub currently reports no conflicts. Discussion/review
prismarine-viewer #514	Maintainer authorized	Green	No	rom1504 approved resolving conflicts and merging. GitHub currently reports no conflicts. Discussion/review
mineflayer #4085	Updated; re-review	Green	No	rom1504 requested keeping loader.js generic; U9G moved the counter back into a sequence plugin in 63411b6. Author response awaits review; earlier ready verdict was for a different head. Discussion/review
mineflayer #4104	Updated; re-review	Red	No	U9G answered the real-client test request and changed resource-pack reply handling in 5bd171c. Author reports passing tests; current CI fails the 1.18.2 scoreboard-reset assertion. Re-review changes and resolve/rerun CI. Discussion/review
prismarine-item #188	Updated; re-review	Green	Unknown	U9G reports fixing pre-1.13 literal names in df76ff7 with 125 passing tests. CI is green; the new head needs review rather than carrying forward the old defect as confirmed. Discussion/review
node-minecraft-protocol #1525	Maintainer test request	Green	No	rom1504 requested using the real client. Head unchanged; that request now takes precedence over the earlier Astra ready assessment. Discussion/review
mineflayer #4027	Dependency / CI follow-up	Green	No	rom1504 confirmed item #184 is not ready; its release remains the prerequisite. Discussion/review
mineflayer #4062	Dependency / CI follow-up	Green	No	Data #1273 is merged; the existing requirement is to verify a published minecraft-data dependency containing the feature/schema changes. Discussion/review
mineflayer #4093	Dependency / CI follow-up	Green	No	Still depends on #4063 and its mounted-input repair; head unchanged. Discussion/review
mineflayer #4094	Dependency / CI follow-up	Red	Unknown	Still based on split-4066/sequence-counter; coordinate #4085 and rerun failed CI. Head unchanged. Discussion/review
mineflayer #4115	Dependency / CI follow-up	Red	No	Head unchanged; previous geometry checks passed, but failed CI still needs follow-up. Discussion/review
mineflayer #4118	Dependency / CI follow-up	Socket only	Yes	Head unchanged; conflict resolution, normal test CI and the existing entity-ownership discussion remain. Full extraction was not required by our earlier review. Discussion/review
mineflayer #4127	Dependency / CI follow-up	Red	No	Head unchanged; confirm the released cursor-schema dependency and rerun failed CI. Discussion/review
node-minecraft-protocol #1522	Dependency / CI follow-up	Green	No	Head unchanged; documented 1.21.8 stored-cookie schema prerequisite still needs coordination/release verification. Discussion/review
prismarine-item #184	Dependency / CI follow-up	Red	No	rom1504 flagged failing CI. ProtoDef #177 is still open (and conflicting); dependency/release and CI follow-up remain. Discussion/review
mineflayer #4063	Existing defect	Socket only	Yes	Head unchanged; mounted-input suppression from the September 19 review remains. Also conflicts with master. Discussion/review
mineflayer #4064	Existing defect	Green	No	Head unchanged; healthy-respawn/player_loaded finding remains. Discussion/review
mineflayer #4065	Existing defect	Green	No	Head unchanged; supplied-client settings ownership finding remains. Discussion/review
mineflayer-pathfinder #382	Existing defect	Red	Unknown	Head unchanged; faceAt timeout still reports success in the reviewed implementation. CI also remains red. Discussion/review

The eight maintainer-authorized rows reflect explicit comments, not formal GitHub APPROVED reviews. Four currently have no merge conflicts: Mineflayer #4107 and viewer #506/#513/#514. Mineflayer #4107 still shows GitHub’s review requirement; “no conflicts” does not mean every branch-protection requirement is satisfied. The other four authorized PRs still conflict.

The new #4104 head has a failed CI job on the 1.18.2 scoreboard-reset assertion. That identifies the failing test; it does not establish the failure’s root cause. Missing normal test jobs on #376/#4118 are shown as “Socket only”, not a successful test build.

Other 19 open U9G PRs outside the previous 33-row table

These are a current GitHub inventory, not fresh readiness assessments. Their heads are unchanged from the September 19 census.

PR	Checks	Conflicts	Status note
minecraft-data #1292	Green	No	Open; outside this review-table refresh.
minecraft-data #1302	Red	Yes	Open; outside this review-table refresh.
mineflayer #4021	Green	No	Open; outside this review-table refresh.
mineflayer #4074	Red	No	Open; outside this review-table refresh.
mineflayer #4075	Red	No	Open; outside this review-table refresh.
mineflayer #4081	Red	No	Open; outside this review-table refresh.
mineflayer #4086	Green	No	Open; outside this review-table refresh.
mineflayer #4087	Green	No	Open; outside this review-table refresh.
mineflayer #4089	Red	Yes	Open; outside this review-table refresh.
mineflayer #4103	Green	Yes	Open; outside this review-table refresh.
mineflayer #4111	Green	Yes	Open; outside this review-table refresh.
mineflayer #4114	Green	No	Open; outside this review-table refresh.
mineflayer #4116	Green	No	Open; outside this review-table refresh.
mineflayer #4126	Green	Yes	Open; outside this review-table refresh.
node-minecraft-protocol #1526	Green	No	Open; outside this review-table refresh.
ProtoDef #65	No checks	No	Hashing prerequisite for item #184; still open.
node-protodef #177	Green	Yes	Hashing prerequisite for item #184; still open.
node-protodef #179	Green	No	Open; outside this review-table refresh.
node-protodef #180	Green	Yes	Open; outside this review-table refresh.

Skills used: prismarine-review kept merged status separate from readiness, incorporated later human instructions, and prevented earlier ready/blocked verdicts from being silently applied to changed heads. No new PR reviews or code changes were made in this status pass.