Skip to content

Harden v2 backward compatibility - #82

Draft
instafluff wants to merge 13 commits into
v2from
agent/v2-backcompat
Draft

Harden v2 backward compatibility#82
instafluff wants to merge 13 commits into
v2from
agent/v2-backcompat

Conversation

@instafluff

Copy link
Copy Markdown
Owner

What changed

  • adds a dedicated v1 compatibility boundary for the public ComfyJS bundle
  • restores legacy chat delivery semantics for highlighted messages, channel-point messages, /me actions, and @mention !command commands
  • freezes legacy onJoin, onPart, and onRaid payload shapes
  • preserves legacy cheer callback behavior without also firing onChat
  • adds regression tests for these compatibility contracts
  • adds secret-free CI across Node 20, 22, and 24 with typecheck, tests, build, and package verification

Why

The existing v2 branch had strong parser/unit coverage, but it did not have an explicit compatibility boundary. Several later changes had drifted from v1 behavior even after parity fixes were documented. This PR makes backward compatibility executable rather than relying on a checklist.

Next in this PR

  • expand the compatibility oracle across the rest of the v1 API and callback payloads
  • verify package/browser/CommonJS import surfaces
  • add modern EventSub events through additive APIs without mutating legacy callbacks
  • add live/captured differential validation

Draft while compatibility coverage is being completed.

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