Skip to content

stale-impossibility: main( ) is not a control method - #184

Merged
oblomov-dev merged 1 commit into
mainfrom
claude/viele-kleine-verbesserungen-9lzh6d
Sep 4, 2026
Merged

stale-impossibility: main( ) is not a control method#184
oblomov-dev merged 1 commit into
mainfrom
claude/viele-kleine-verbesserungen-9lzh6d

Conversation

@oblomov-dev

Copy link
Copy Markdown
Member

The probe reads a deviation's prose for someMethod( ) and asks the frontend's allowlist whether that method is refused. Four names in these sidecars are not control methods at all and can never be on that list, so a claim mentioning one produces a hit that cannot be real:

  • main( )z2ui5_if_app~main, the ABAP entry point every port implements and the notes discuss constantly
  • render( ) and stringify( )z2ui5_cl_ui5_view_builder's own (and stringify is also JSON's, which is how app 447's note mentions it)
  • factory( ) — opens every chain

That was the whole standing hit list. App 351's note explains that Invalidate is denied, and that its branch used to be a no-op because main_end only sends a model when main( ) changed one — and the probe asked whether main is denied. Six lines of reading, every sweep, for a question with no answer.

before:  622 ports scanned, 1 deviation(s) to re-read
after:   622 ports scanned, 0 deviation(s) to re-read

Re-verified against a synthetic defect, as the probe README asks: a deviation claiming "no abap2UI5 equivalent" while naming setBusy( ) on a control is still reported. The other fourteen corpus probes were run alongside and all report zero.

🤖 Generated with Claude Code

https://claude.ai/code/session_01CD8XM2xmi9NHb17JHP67nZ


Generated by Claude Code

The probe reads a deviation's prose for `someMethod( )` and asks the frontend's
allowlist whether that method is refused. Four names in these sidecars are not
control methods at all and can never be on that list, so a claim that mentions
one produces a hit that cannot be real: `main( )` is z2ui5_if_app~main, the
ABAP entry point every port implements and the notes discuss constantly;
`render( )` and `stringify( )` are z2ui5_cl_ui5_view_builder's own (and
`stringify` is also JSON's, which is how app 447's note mentions it);
`factory( )` opens every chain.

That was the whole standing hit list: app 351's note explains that Invalidate
is denied and that the branch used to be a no-op because main_end only sends a
model when main( ) changed one - and the probe asked whether `main` is denied.
Six lines of reading, every sweep, for a question with no answer.

622 ports, 0 to re-read now. Re-verified against a synthetic defect: a
deviation naming `setBusy( )` on a control is still reported.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CD8XM2xmi9NHb17JHP67nZ
Copilot AI lite review requested due to automatic review settings September 4, 2026 23:23

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@oblomov-dev
oblomov-dev merged commit 9ad8ef6 into main Sep 4, 2026
19 checks passed
@oblomov-dev
oblomov-dev deleted the claude/viele-kleine-verbesserungen-9lzh6d branch September 4, 2026 23:36
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.

3 participants