Skip to content

Count the classes and src/01 ports the README promises as they are - #265

Merged
oblomov-dev merged 1 commit into
mainfrom
claude/beautiful-turing-wxr303
Oct 11, 2026
Merged

oblomov-dev merged 1 commit into
mainfrom
claude/beautiful-turing-wxr303

Conversation

@oblomov-dev

Copy link
Copy Markdown
Member

Round-4 review (iteration 3: docs drift).

Fix

  • README, "Try it in 60 seconds" step 2 said 637 classes in 14 packages and 416 ports in src/01. The tree holds 642 classes in 15 packages, and src/01 holds 401 ports - the number STATUS.md's generated state block gives as well (src/02 stays at 221). This is the line a user checks an abapGit pull against.

Reviewed, nothing to change

The round also re-read today's merged change (app 192's SWITCH downport fix), ran npm run check on main (green), and hunted for user-visible defects the gates do not decide: unguarded table reads and CONV i( ) on event arguments, JS getMonth() without + 1, event arguments compared to true/false strings (the framework turns a boolean argument into X/empty), parameters named like a method addition, .clas.xml sidecars, and several generated ports read against their demo kit controllers. None of it turned up a defect.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Te3BRpRpxNhZAYnrQL6Gdt


Generated by Claude Code

The install step said 637 classes in 14 packages and 416 ports in src/01.
The tree holds 642 classes in 15 packages, and src/01 holds 401 ports -
the number STATUS.md's generated state block gives as well. The numbers
are what a user checks the abapGit pull against.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Te3BRpRpxNhZAYnrQL6Gdt
@oblomov-dev
oblomov-dev merged commit 189e69e into main Oct 11, 2026
21 checks passed
@oblomov-dev
oblomov-dev deleted the claude/beautiful-turing-wxr303 branch October 11, 2026 18:33
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.

2 participants