Skip to content

Reimpl/AICState development - #262

Draft
gynt wants to merge 27 commits into
reimpl/AICState-100from
reimpl/AICState-leftover
Draft

gynt wants to merge 27 commits into
reimpl/AICState-100from
reimpl/AICState-leftover

Conversation

@gynt

@gynt gynt commented Oct 14, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@gynt
gynt changed the base branch from main to reimpl/AICState-100 October 14, 2026 07:50
@gynt
gynt marked this pull request as draft October 14, 2026 08:03
@gynt
gynt force-pushed the reimpl/AICState-100 branch from 959b9a9 to a8f7dae Compare October 14, 2026 08:15
gynt and others added 6 commits October 14, 2026 14:39
…l/PathFindingState

Brings the tools-only work from the reimpl/PathFindingState branch to main:

- undirwalk.py / undirwalk4.py rebuild the neighbour-direction loops Ghidra
  renders as two pointers walked in lockstep.
- unoffset_matrix.py resolves hand-computed byte offsets into matrix fields.
- unguarded_do.py, docount.py and early_return.py undo three more decompiler
  shapes (guarded do-while, counted do-while, one-big-if bodies).
- diff_signed.py reports comparisons whose signedness differs from the original.

The shared files (common.py, build_quiet.py, syntax_check.py, quick_diff.py,
openshcmcp.py and the README) already carry main's newer versions of the same
work - the PDB retries, --cache/--quiet, quick_diff's -s slot mode - so those
keep main's content and only the README gains rows for the new scripts.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…t) index cast

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0179pNuXVNcvqm3mQfQaRDMs
…rm merges

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0179pNuXVNcvqm3mQfQaRDMs
@gynt
gynt force-pushed the reimpl/AICState-leftover branch from bd0ce24 to 8251a29 Compare October 15, 2026 22:08
gynt and others added 20 commits October 16, 2026 02:56
The header declares the parameter as int; the PlayerID typedef the
definitions used is not visible there and broke the build.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JP48NBCkB9vy6M7zLArtgo
aiBuyAndSellGoods only differs in its tail-jump targets; with the
sellExcessGoods and buyRequiredGoods resolvers enabled it matches 100%.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JP48NBCkB9vy6M7zLArtgo
…nabled

computeAttackPossibilities, playNervous2BikFromPlayer and buyRequiredGoods
were recorded before the global data resolvers were switched on.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JP48NBCkB9vy6M7zLArtgo

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant