Skip to content

feat: state machine overhaul — begin(), 5 active states, symmetric handshake - #20

Merged
petesramek merged 3 commits into
mainfrom
copilot/review-codebase-assessment
Mar 23, 2026
Merged

petesramek merged 3 commits into
mainfrom
copilot/review-codebase-assessment

Conversation

Copilot AI commented Mar 23, 2026 •

Copy link
Copy Markdown
Contributor
  • MessageType.h — Debug = 'g' → Log = 'L'; string "Debug" → "Log"
  • src/protocol/internal/DebugMessage.h → LogMessage.h (git mv); DebugMessage→LogMessage, debugmessage_set_text→logmessage_set_text, DEBUG_TEXT_CAPACITY→LOG_TEXT_CAPACITY
  • TinyLink.h — include path, struct usage, helper call, wire-format comment updated
  • TinyProtocol.h — comment updated
  • LogLevel.h — wire-frame reference 'g' → 'L' in comment
  • test/test_debugmessage.cpp → test/test_logmessage.cpp (git mv); all symbols renamed; truncation test validates code field isolation
  • test/test_message_type.cpp — MessageType::Debug → MessageType::Log, 'g' → 'L', "Debug" → "Log"
  • test/test_runner.cpp — forward declaration + call updated
  • All 150 tests pass; code review clean; CodeQL: 0 alerts

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Copilot AI and others added 2 commits March 23, 2026 09:51
…FRAME_COMPLETE/AWAITING_ACK

Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
Agent-Logs-Url: https://github.com/petesramek/tiny-link/sessions/4d1aacd5-7d22-48ac-85cb-85de1747ec8c
…geType::Debug='g'→Log='L')

Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
Agent-Logs-Url: https://github.com/petesramek/tiny-link/sessions/75aadd81-2119-4deb-b514-c225b9776730
Copilot AI requested a review from petesramek March 23, 2026 10:25
@petesramek
petesramek marked this pull request as ready for review March 23, 2026 10:26
@petesramek
petesramek merged commit 8446d35 into main Mar 23, 2026
4 checks passed
@petesramek
petesramek deleted the copilot/review-codebase-assessment branch March 23, 2026 12:20
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