Skip to content

Improve README and refactor Winsock TCP handling - #2

Merged
papanda925 merged 1 commit into
mainfrom
codex/readme-winsock-refactor
Sep 2, 2026
Merged

papanda925 merged 1 commit into
mainfrom
codex/readme-winsock-refactor

Conversation

@papanda925

@papanda925 papanda925 commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace the minimal README with Japanese-first setup, usage, limitations, troubleshooting, and official references
  • correct Winsock API declarations for VBA7 32-bit and 64-bit Office by using LongPtr for socket handles
  • use the native four-parameter send signature
  • close accepted client sockets after each request and guard all cleanup paths
  • stop closing the workbook and the entire Excel application automatically
  • add repository-level static validation and a Windows GitHub Actions workflow

Validation

  • CP932 source round-trip validation passed
  • VBA Sub / Function structure counts are balanced
  • static checks confirm LongPtr handles, ws2_32.dll, and removal of automatic Excel shutdown
  • GitHub Actions completed successfully
  • Windows Excel runtime verification completed on 2026-09-02
  • MainForMultiProcess, testHELLO, testElse, and testQUIT completed successfully

Scope

This remains a low-level TCP/Winsock learning sample. HTTP, WebSocket, and LLM API functionality are intentionally out of scope.

@papanda925
papanda925 marked this pull request as ready for review September 2, 2026 14:24
@papanda925
papanda925 merged commit 89bb710 into main Sep 2, 2026
2 checks passed
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