Merge pull request #1196 from JuliaLang/json-int

Type-assert Int64's from JSON

Author: JamesWrigley

.github/workflows/CI.yml

@@ -31,8 +31,17 @@ jobs:
       - uses: julia-actions/setup-julia@v2
         with:
           version: ${{ matrix.julia-version }}
+          arch: ${{ matrix.arch }}
 
       - uses: julia-actions/cache@v2
+
+      # Installing Python on x86 is difficult and IJulia otherwise won't
+      # precompile, so we just remove PythonCall from the test deps so that the
+      # extension will never be built.
+      - name: Remove PythonCall on x86
+        if: matrix.arch == 'x86'
+        run: julia --project=test -e 'using Pkg; Pkg.rm("PythonCall")'
+
       - uses: julia-actions/julia-buildpkg@v1
       - uses: julia-actions/julia-runtest@v1
 

docs/src/_changelog.md

@@ -7,6 +7,11 @@ CurrentModule = IJulia
 This documents notable changes in IJulia.jl. The format is based on [Keep a
 Changelog](https://keepachangelog.com).
 
+## Unreleased
+
+### Fixed
+- Fixed support for 32bit systems ([#1196]).
+
 ## [v1.31.0] - 2025-10-13
 
 ### Added

src/handlers.jl

@@ -124,7 +124,7 @@ request](https://jupyter-client.readthedocs.io/en/latest/messaging.html#completi
 """
 function complete_request(socket, kernel, msg)
     code = msg.content["code"]::String
-    cursor_chr = msg.content["cursor_pos"]::Int
+    cursor_chr = msg.content["cursor_pos"]::Int64
     cursorpos = chr2ind(msg, code, cursor_chr)
     # Ensure that `cursorpos` is within bounds, Jupyter may send a position out
     # of bounds when autocompletion is enabled.
@@ -375,7 +375,7 @@ request](https://jupyter-client.readthedocs.io/en/latest/messaging.html#introspe
 function inspect_request(socket, kernel, msg)
     try
         code = msg.content["code"]::String
-        cursor_pos = msg.content["cursor_pos"]::Int
+        cursor_pos = msg.content["cursor_pos"]::Int64
         s = get_token(code, chr2ind(msg, code, cursor_pos))
         if isempty(s)
             content = Dict("status" => "ok", "found" => false)

test/runtests.jl

@@ -4,14 +4,21 @@ import IJulia
 
 const TEST_FILES = [
     "install.jl", "comm.jl", "msg.jl", "execute_request.jl", "stdio.jl",
-    "inline.jl", "completion.jl", "kernel.jl"
+    "inline.jl", "completion.jl"
 ]
 
 for file in TEST_FILES
     println(file)
     include(file)
 end
 
+# Python is well-nigh impossible to install on 32bit
+if Sys.WORD_SIZE != 32
+    include("kernel.jl")
+else
+    @warn "Skipping the Kernel tests on 32bit"
+end
+
 @testset "Aqua.jl" begin
     # Note that Pkg and Conda are loaded lazily
     Aqua.test_all(IJulia; stale_deps=(; ignore=[:Pkg, :Conda]))