[kixelated]

Summary

Plans the follow-up of moq-dev/rust2#662, now queued: Array references from slices and cells [S]. It covers TryFrom<&mut [T]> for &mut [T; N] (slice.try_into()), Cell::as_array_of_cells, and convert::identity of a &mut [T; N], each lending the array where it is. Today they're unsupported calls, and try_into this way is common in parsers.

Decisions

• ✅ Plan it in m3, decided with the maintainer on 2026-11-07 when they queued feat(ts)!: parts of arrays written where they are, without arrayView #662.
• Dropped: feat(ts)!: parts of arrays written where they are, without arrayView #662's other follow-up, the raw-pointer fixture move. That's the raw-pointer model working as designed.

🤖 Generated with Claude Code

(Written by Claude Opus 5.5 with high effort)

Review

Rewritten by quest review after each round.

• Reviewer: Codex, gpt-6.1-sol, low effort
• Session: 01a1b8c7-efde-7380-8387-94325e51756f, round 1
• Head: 9276ad5
• Base: origin/main at 8254acb
• Score: 10/10, The diff adds a scoped planning document and a valid quest index link, with no executable changes or actionable defects.
• Findings: none

[pullfrog]

  ^{｜ Review this ➔}

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 22 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: f3d1d3fb-0798-456e-8d20-4938cef8017e

Commits

Reviewing files that changed from the base of the PR and between 8254acb and 9276ad5.

Files selected for processing: 2

• quest/m3/README.md
• quest/m3/ts-array-ref-conversions.md

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[kixelated]

Automated review by review (OpenAI)

Reviewed head: 9276ad5b3ef94318df7287d41084e0dd375bce97
Base: 8254acb90050f21f442be92f93e2caaaf3685dff

No actionable findings in the complete two-file, documentation-only diff. The new quest is linked correctly from the m3 index, captures the safe-API follow-ups documented in #662, and keeps the implementation within the existing slice representation and m3 unsafe policy. Requiring differential tests against native Rust is the right validation direction for preserving borrowing and mutation semantics.

Scope: static review through GitHub of the full diff, linked planning documents, repository review guidance, and #662's documented follow-ups. No tests, builds, or quest checks were run; the future implementation is not validated by this review. Submitted as COMMENT, not approval.

(Written by review (OpenAI))