Skip to content

Track #475 bugs in KNOWN_ISSUES.md#476

Merged
aallan merged 1 commit into
mainfrom
track-475-known-issues
Apr 16, 2026
Merged

Track #475 bugs in KNOWN_ISSUES.md#476
aallan merged 1 commit into
mainfrom
track-475-known-issues

Conversation

@aallan

@aallan aallan commented Apr 16, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

One-line addition to the Bugs table for #475 (aggregate tracking issue for 10 pre-existing WASM call translator bugs surfaced during review of #474).

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation
    • Updated known issues documentation with identified issues affecting WASM call translator functionality.

@aallan
Track WASM call translator bugs from #475 in KNOWN_ISSUES.md
15b4fb3 
Adds the aggregate tracking issue for 10 pre-existing bugs
surfaced during review of the calls.py decomposition (#474).

Co-Authored-By: Claude <noreply@anthropic.invalid>
@coderabbitai

coderabbitai Bot commented Apr 16, 2026 •
edited
Loading

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 185fcc4e-10f4-419d-9fe0-5c50049b0375

📥 Commits

Reviewing files that changed from the base of the PR and between ac718f2 and 15b4fb3.

📒 Files selected for processing (1)
  • KNOWN_ISSUES.md
📝 Walkthrough

Walkthrough

This pull request adds a documentation entry to KNOWN_ISSUES.md cataloguing 10 pre-existing bugs within the WASM call translators, covering problems such as INT64_MIN string conversion, type narrowing in slice translation, missing bounds checks, and exception handler typing issues.

Changes

Cohort / File(s) Summary
WASM Call Translators Known Issues
KNOWN_ISSUES.md		 Added new bug entry for issue #475 documenting 10 pre-existing defects in WASM call translators, including INT64_MIN to_string conversion, i64→i32 narrowing in slice translation, missing char_code bounds check, exception handler result typing, Map lowering for nested parameterised types, URL parsing round-trip behaviour, base64 padding validation, numeric parsing whitespace handling, and float fractional carry propagation.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and accurately summarises the primary change: documenting issue #475 bugs in KNOWN_ISSUES.md.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch track-475-known-issues

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov

codecov Bot commented Apr 16, 2026 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.35%. Comparing base (ac718f2) to head (15b4fb3).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #476   +/-   ##
=======================================
  Coverage   90.35%   90.35%           
=======================================
  Files          50       50           
  Lines       19548    19548           
  Branches      225      225           
=======================================
  Hits        17663    17663           
  Misses       1881     1881           
  Partials        4        4
Flag Coverage Δ
javascript 50.67% <ø> (ø)
python 95.28% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aallan aallan merged commit e9739d0 into main Apr 16, 2026
19 checks passed
@aallan aallan deleted the track-475-known-issues branch April 16, 2026 20:01
This was referenced May 5, 2026
Merged
Merged
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.

1 participant

@aallan