Skip to content
This repository was archived by the owner on Apr 8, 2026. It is now read-only.

Align SkyHook event layout and enums; fix time/label mapping; add READMEs#6

Merged
2228293026 merged 2 commits into
masterfrom
codex/refactor-skyhook-to-use-internal-game-system-590zhf
Mar 7, 2026
Merged

Align SkyHook event layout and enums; fix time/label mapping; add READMEs#6
2228293026 merged 2 commits into
masterfrom
codex/refactor-skyhook-to-use-internal-game-system-590zhf

Conversation

@2228293026

Copy link
Copy Markdown
Member

Motivation

  • Ensure the mod's SkyHook compatibility layer exactly matches the in-game SkyHookEvent layout and types to avoid mismatches when marshalling or comparing events.
  • Remove ambiguity by qualifying enum usages so the async input consumer references the engine SkyHook enums unambiguously.
  • Provide user and developer documentation with English and Chinese README files.

Description

  • Replace local EventType/KeyLabel usages with global::SkyHook.EventType and global::SkyHook.KeyLabel and qualify enum comparisons in AsyncInputManager to avoid ambiguous references.
  • Redefine SkyHookEvent as StructLayout(LayoutKind.Explicit) with FieldOffset annotations to match the game structure and add GetTimeInTicks plus a corrected time conversion routine in ConvertTicksToSkyHookTime.
  • Replace the previous GetKeyLabel implementation with a compact switch expression that maps key codes to global::SkyHook.KeyLabel values and adjust several label names to match the SkyHook API (Enter, ArrowLeft/ArrowUp/ArrowRight/ArrowDown, Unknown, etc.).
  • Add documentation files README.md and README.en.md describing features, configuration and build notes for the project.

Testing

  • Built the project (Release) with the normal .NET build process (msbuild / Visual Studio) and confirmed the compilation succeeds.
  • No automated unit tests were present or run as part of this change.

Codex Task

@2228293026 2228293026 merged commit cc6dd37 into master Mar 7, 2026
@2228293026 2228293026 deleted the codex/refactor-skyhook-to-use-internal-game-system-590zhf branch March 7, 2026 08:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant