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

fix: apply naval academy runtime catch-up on login - #233

Merged
ggmolly merged 4 commits into
mainfrom
feat/timed-packet-audit
Feb 22, 2026
Merged

fix: apply naval academy runtime catch-up on login#233
ggmolly merged 4 commits into
mainfrom
feat/timed-packet-audit

Conversation

@ggmolly

@ggmolly ggmolly commented Feb 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add naval academy runtime persistence for canteen and merchant levels, renovation windows, and last-collection timestamps.
  • Apply login catch-up so oil and coin accrual is granted from stored collection timestamps using level production/store rules.
  • Return runtime-backed canteen and merchant levels/timers in the resource info packet with level normalization against available templates.

Changes

  • add internal/orm/naval_academy_runtime.go to load/create/save per-commander naval academy runtime state in config storage.
  • update internal/answer/playerops/last_login.go to run naval academy catch-up before login state is sent.
  • update internal/answer/playerops/resources_info.go to source canteen/merchant levels and renovation timers from runtime state.
  • add internal/answer/playerops/naval_academy_runtime.go with template parsing, level clamping, renovation completion handling, and accrual computation that pauses during renovation.
  • extend internal/answer/config_handlers_test.go with coverage for runtime-backed timers, level normalization, and login catch-up accrual.

@ggmolly ggmolly added type/bug Defect or incorrect behavior area/inventory Items/currency area/save Serialization/persistence labels Feb 22, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: b5eb8032fc

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread internal/answer/playerops/naval_academy_runtime.go Outdated
@ggmolly
ggmolly merged commit 70fa028 into main Feb 22, 2026
1 check passed
@ggmolly
ggmolly deleted the feat/timed-packet-audit branch February 22, 2026 13:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area/inventory Items/currency area/save Serialization/persistence type/bug Defect or incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant