Forth by luginf · Pull Request #2947 · nesbox/TIC-80

luginf · 2026-05-31T14:26:07Z

adding Forth langages to TIC-80 (using pforth)

luginf · 2026-05-31T14:31:51Z

(made with the help of Claude code)

pforth cell_t = uintptr_t, so the pre-compiled dictionary is cell-size-dependent (64-bit on x86-64, 32-bit on WASM). Loading a 64-bit dictionary in a 32-bit runtime corrupts all addresses and freezes the HTML player. - cmake/pfdicdat_32.h: new 32-bit dictionary (generated with gcc -m32) - cmake/forth.cmake: select pfdicdat_32.h when CMAKE_SIZEOF_VOID_P == 4

ddelemeny · 2026-06-01T15:46:22Z

Note: OP had the courtesy to open a discussion (#2938) regarding the project's AI contribution policy before vibe-coding this payload.

Resolving the policy matter should probably be considered a prerequisite to reviewing this PR.

MineRobber9000

I don't know enough about pforth to critique the actual implementation of the API, but there are definitely some spots in here that I am concerned about.

MineRobber9000 · 2026-06-01T15:38:01Z

+            message(STATUS "Forth: copying bundled ${_PFORTH_DICDAT_BUNDLED} to ${PFORTH_DICDAT}")
+            configure_file(${_PFORTH_DICDAT_BUNDLED} ${PFORTH_DICDAT} COPYONLY)
+        else()
+            message(STATUS "Forth: pfdicdat.h not found — bootstrapping pforth to generate it...")


If we can use pforth to bootstrap the pfdicdat.h generation, why are we bundling a version of this data anyways? What happens if pforth updates in a way that adds more stuff to the dictionary, and now our bundled data is out of date?

@MineRobber9000 that's a good question. I've initially let it generate the pfdicdat.h and pfdicdat_32.h from pforth but there was a problem with some targets (nintendo3D and rpi) do I've included those headers but it's not ideal.

It should be fixed now. I've tested the nintendo 3DS build and it's working fine.

MineRobber9000 · 2026-06-01T15:40:54Z

+\ license: MIT License
+\ script: forth
+\
+\ -- TIC-80 Forth API --------------------------------------------------


You don't need to leave this API reference in the code.

MineRobber9000 · 2026-06-01T15:42:52Z

+;
+
+\ -- Demo 2 : sprite + arrow keys -------------------------------------
+: demo02


This is the only one you actually need to have. The others are all just clutter that a user would have to delete before starting their own game.

sure. I've removed all that, including the API references and additional examples. Do you think it should be better to include such informations in an additional file in the demos/ folder under a new name, or leave this to the wiki, or even somewhere else? I see in the demos folder some lua samples like fire.lua or tetris.lua so I don't know

MineRobber9000 · 2026-06-01T15:48:13Z

+
+: BOOT
+  96 px !  24 py !
+  S" Forth BOOT" 12 TRACE


Remove this, no other language does this. (EDIT: to clarify, since GH shows a seemingly random collection of lines and not the one you explicitly select in the interface, this is referring to tracing "Forth BOOT" in the BOOT callback.)

ok. I've just kept the BOOT procedure for initialising the variables, and kept the "demo01" word outside of the TIC loop, so people can just comment the demo and start creating new words.

MineRobber9000 · 2026-06-01T15:53:41Z

@@ -0,0 +1,129 @@
+// MIT License


This should also go in forth.c. In general, all languages should be one C file apiece; the only exception are the Lua languages (Lua, MoonScript, Yue, and Fennel), since they all share the same plumbing underneath and only differ in how they're initialized.

it has been merged into the forth.c

removing cmake/pfdicdat.h and cmake/pfdicdat_32.h from tree updating cmake/forth.cmake for bootstraping pfdicdat

…3ds, rpi)

MineRobber9000 · 2026-06-02T21:37:19Z

How did you manage to chmod +x the whole repo?

luginf · 2026-06-02T22:07:53Z

How did you manage to chmod +x the whole repo?

oops, long story short I needed to edit some files from another (local) account (on my computer). I can try to revert the 755 to 644 if you wish, but that will trigger a new build for nothing.

MineRobber9000 · 2026-06-03T07:44:56Z

You definitely should undo it. I can't review the PR in this state since I'd have to scroll through basically every file in the repo just to see what you actually changed.

luginf · 2026-06-03T08:11:36Z

You definitely should undo it. I can't review the PR in this state since I'd have to scroll through basically every file in the repo just to see what you actually changed.

Ok, i've fixed it, I'm just working on another problem (wrong/misleading message during a stack underflow) and I'll commit that ASAP

fix "stack underflow" message

MineRobber9000 · 2026-06-03T10:06:43Z

+: demo01
+  0 BTN IF  py @ 1- py ! THEN
+  1 BTN IF  py @ 1+ py ! THEN
+  2 BTN IF  px @ 1- px ! THEN
+  3 BTN IF  px @ 1+ px ! THEN
+  13 CLS
+  1 px @ py @ 14 3 0 0 2 2 SPR
+  S" Hello Forth!" 84 84 15 FALSE 1 FALSE PRINT DROP
+;
+
+: TIC
+demo01
+;


Suggested change

: demo01

0 BTN IF py @ 1- py ! THEN

1 BTN IF py @ 1+ py ! THEN

2 BTN IF px @ 1- px ! THEN

3 BTN IF px @ 1+ px ! THEN

13 CLS

1 px @ py @ 14 3 0 0 2 2 SPR

S" Hello Forth!" 84 84 15 FALSE 1 FALSE PRINT DROP

;

: TIC

demo01

;

: TIC

0 BTN IF py @ 1- py ! THEN

1 BTN IF py @ 1+ py ! THEN

2 BTN IF px @ 1- px ! THEN

3 BTN IF px @ 1+ px ! THEN

13 CLS

1 px @ py @ 14 3 0 0 2 2 SPR

S" Hello Forth!" 84 84 15 FALSE 1 FALSE PRINT DROP

;

If the user wants to write their own code, they'll probably want to just delete the whole demo code, so having it be a separate word is just clutter.

MineRobber9000 · 2026-06-03T10:23:08Z

Also, you forgot to install gcc-multilib in the Android task, which is why that build is failing.

… task.

…ries

fix mac os build by limiting the number of parallel processes

- Remove references to committed cmake/pfdicdat.h and pfdicdat_32.h (deleted in PR nesbox#2947 forth branch) - Document that cmake/forth.cmake now always regenerates pfdicdat.h at configure time via file(REMOVE) + bootstrap - Add critical warning: a stale 32-bit dict in a 64-bit binary causes a silent segfault when selecting "new forth" (root cause documented) - Update platform/cell-size matrix (WASM/3DS/RPI = 32-bit) - Remove now-outdated manual regeneration instructions Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-authored-by: minerobber <milesrobert374@gmail.com>

MineRobber9000

LGTM!

luginf added 4 commits May 31, 2026 11:45

adding forth (using pforth)

4165603

forth now working

e800be7

working example

54641a1

fix demo cart

e473ca5

luginf added 6 commits May 31, 2026 18:43

fix missing pfdicdat from pforth

ab80697

add missing asset

92cd9cc

adding pfdicdat.h

d822781

fix rpi baremetal build

f9cc26e

fix pfdicdat for html build (32 bit)

6352221

luginf force-pushed the forth branch from d176dc5 to 6352221 Compare June 1, 2026 08:23

MineRobber9000 suggested changes Jun 1, 2026

View reviewed changes

luginf added 4 commits June 2, 2026 18:37

fusion of api/forth.h and api/forth_io.h

688734d

removing cmake/pfdicdat.h and cmake/pfdicdat_32.h from tree updating cmake/forth.cmake for bootstraping pfdicdat

fix forth.cmake & workflows to bootstrap on crossplatforms (nintendo …

9b16fdc

…3ds, rpi)

rebuild forthdemo card

091f503

fix 3DS build

787f601

luginf requested a review from MineRobber9000 June 2, 2026 19:03

fix wrong permission

a6643bf

fix "stack underflow" message

MineRobber9000 suggested changes Jun 3, 2026

View reviewed changes

luginf added 4 commits June 3, 2026 16:10

updating forthmark. Clean forthdemo. Fix missing multilib for android…

f9c1ebb

… task.

adding 5 minutes timeout in case of problem during generation of bina…

1551c18

…ries

fix windows build.

8762b9b

fix mac os build by limiting the number of parallel processes

fix windows build for real this time?

7d7b473

luginf added 2 commits June 4, 2026 21:40

update forth.cmake for windows build

0e3a08c

compile in release mode on windows, change also timeout

69452cf

luginf requested a review from MineRobber9000 June 4, 2026 20:53

MineRobber9000 suggested changes Jun 5, 2026

View reviewed changes

Comment thread src/api/forth.c Outdated

Comment thread src/api/forth.c Outdated

Comment thread assets.bat

Comment thread assets.bat

luginf and others added 5 commits June 5, 2026 08:25

Update src/api/forth.c

25588aa

Co-authored-by: minerobber <milesrobert374@gmail.com>

Update src/api/forth.c

896a8a8

Co-authored-by: minerobber <milesrobert374@gmail.com>

Update assets.bat

6389821

Co-authored-by: minerobber <milesrobert374@gmail.com>

Update assets.bat

ef3fd39

Co-authored-by: minerobber <milesrobert374@gmail.com>

adding forthmark.tic.dat

6e47191

MineRobber9000 approved these changes Jun 6, 2026

View reviewed changes

Uh oh!

Conversation

luginf commented May 31, 2026

Uh oh!

luginf commented May 31, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

ddelemeny commented Jun 1, 2026

Uh oh!

MineRobber9000 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MineRobber9000 Jun 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

MineRobber9000 commented Jun 2, 2026

Uh oh!

luginf commented Jun 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

MineRobber9000 commented Jun 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

luginf commented Jun 3, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

MineRobber9000 commented Jun 3, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

MineRobber9000 left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

luginf commented May 31, 2026 •

edited

Loading

MineRobber9000 Jun 1, 2026 •

edited

Loading

luginf commented Jun 2, 2026 •

edited

Loading

MineRobber9000 commented Jun 3, 2026 •

edited

Loading