Track per-team solve attempts with timestamped correctness state by Copilot · Pull Request #30 · csboo/apollo

Copilot · 2026-04-11T14:20:53Z

This PR updates team progress tracking from a solved-set model to an append-only attempt history, recording both correctness and attempt time for each submission.

What changed

Replaced per-team solved puzzle storage with per-team Vec<SolveAttempt>.
Added SolveAttempt { puzzle_id, attempted_at, correct }.
Switched attempt timestamps to std::time::SystemTime (dropped jiff usage).
Updated submission flow to log every attempt (correct and incorrect).
Preserved existing behavior:
- Incorrect submissions are rejected.
- Already-solved puzzles cannot be solved again.
Added/kept helper logic to derive solved status from attempt history (team_has_solved_puzzle).
Kept state persistence compatible with the new team progress shape.

Validation

Rebased branch onto latest house-keeping (stacked PR base).
Ran make check successfully after updates.

Agent-Logs-Url: https://github.com/csboo/apollo/sessions/f6216e44-f616-4a05-ae69-cbf6cddb62ef Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

jarjk · 2026-04-13T12:00:18Z

@dependabot rebase

jarjk · 2026-04-13T12:00:43Z

@copilot rebase

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Copilot · 2026-04-13T12:04:54Z

@copilot rebase

Rebased as requested; branch is now updated at 6b78ab7.

jarjk · 2026-04-22T10:18:36Z

@copilot rebase

jarjk · 2026-04-23T19:17:35Z

@copilot drop jiff, use systemtime

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Agent-Logs-Url: https://github.com/csboo/apollo/sessions/f6216e44-f616-4a05-ae69-cbf6cddb62ef Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

…s' into copilot/store-solve-attempt-times # Conflicts: # Cargo.toml # src/backend/endpoints.rs # src/backend/models.rs Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Copilot · 2026-04-23T19:27:06Z

@copilot rebase

Done — rebased and updated branch at 7b5c493.

@copilot drop jiff, use systemtime

Done — removed jiff usage and switched solve-attempt timestamps to std::time::SystemTime in ffc960d.

jarjk · 2026-04-23T20:36:32Z

@copilot rebase

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Copilot · 2026-04-23T20:42:00Z

@copilot rebase

Done — rebased and updated branch at ea9765e.

jarjk · 2026-04-23T20:46:06Z

    };
    solution.zeroize();
-    is_solution_valid.or_forbidden("érvénytelen megoldás ehhez a feladathoz")?;
+    let is_correct = is_solution_valid;


why new var?

jarjk · 2026-04-23T20:47:46Z

+    use std::time::UNIX_EPOCH;
+
+    #[test]
+    fn marks_puzzle_as_solved_only_for_correct_attempts() {


this doesn't actually test anything, have a look at insta then

Copilot AI assigned Copilot and jarjk Apr 11, 2026

Copilot started work on behalf of jarjk April 11, 2026 14:20 View session

Copilot AI linked an issue Apr 11, 2026 that may be closed by this pull request

store solve-attempt times #23

Open

Copilot AI changed the title ~~[WIP] Add store for solve-attempt times and states~~ Track per-team solve attempts with timestamped correctness state Apr 11, 2026

Copilot AI requested a review from jarjk April 11, 2026 14:32

Copilot finished work on behalf of jarjk April 11, 2026 14:32

Copilot AI and others added 3 commits April 12, 2026 18:29

Initial plan

8590bba

Store timed solve attempts with correctness state

dc9005c

Agent-Logs-Url: https://github.com/csboo/apollo/sessions/f6216e44-f616-4a05-ae69-cbf6cddb62ef Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

refactor: bool instead of enum

c82a9f1

jarjk force-pushed the copilot/store-solve-attempt-times branch from 3091f5e to c82a9f1 Compare April 12, 2026 16:31

Copilot started work on behalf of jarjk April 13, 2026 12:00 View session

jarjk and others added 2 commits April 13, 2026 12:04

chore(readme): inline html -> md (so bun can render it)

cb12c60

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Initial plan

6b78ab7

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Copilot finished work on behalf of jarjk April 13, 2026 12:07

Copilot started work on behalf of jarjk April 23, 2026 19:17 View session

Copilot AI and others added 4 commits April 23, 2026 19:20

Initial plan

606807e

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Store timed solve attempts with correctness state

ffc960d

Agent-Logs-Url: https://github.com/csboo/apollo/sessions/f6216e44-f616-4a05-ae69-cbf6cddb62ef Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Initial plan

e8cabeb

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Merge remote-tracking branch 'origin/copilot/store-solve-attempt-time…

7b5c493

…s' into copilot/store-solve-attempt-times # Conflicts: # Cargo.toml # src/backend/endpoints.rs # src/backend/models.rs Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Copilot finished work on behalf of jarjk April 23, 2026 19:27

jarjk changed the base branch from main to house-keeping April 23, 2026 20:36

Copilot started work on behalf of jarjk April 23, 2026 20:36 View session

jarjk and others added 9 commits April 23, 2026 20:41

ci: build with older glibc for more compatibility

1addf7c

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

chore(make): more thorough clean

0403f29

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

ci: disable wild for now

dbf908c

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

ci: build linux with musl

42c62f9

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

ci: keep gnu

7bb4608

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Initial plan

6f6af4b

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Initial plan

87e3433

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Initial plan

d14e90c

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Initial plan

ea9765e

Co-authored-by: jarjk <118479592+jarjk@users.noreply.github.com>

Copilot finished work on behalf of jarjk April 23, 2026 20:43

jarjk reviewed Apr 23, 2026

View reviewed changes

Base automatically changed from house-keeping to main April 24, 2026 11:28

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Track per-team solve attempts with timestamped correctness state#30

Track per-team solve attempts with timestamped correctness state#30
Copilot wants to merge 18 commits into
mainfrom
copilot/store-solve-attempt-times

Copilot AI commented Apr 11, 2026 •

edited

Loading

Uh oh!

jarjk commented Apr 13, 2026

Uh oh!

jarjk commented Apr 13, 2026

Uh oh!

Copilot AI commented Apr 13, 2026

Uh oh!

jarjk commented Apr 22, 2026

Uh oh!

jarjk commented Apr 23, 2026

Uh oh!

Copilot AI commented Apr 23, 2026 •

edited

Loading

Uh oh!

jarjk commented Apr 23, 2026

Uh oh!

Copilot AI commented Apr 23, 2026

Uh oh!

jarjk Apr 23, 2026

Uh oh!

jarjk Apr 23, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

Copilot AI commented Apr 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What changed

Validation

Uh oh!

jarjk commented Apr 13, 2026

Uh oh!

jarjk commented Apr 13, 2026

Uh oh!

Copilot AI commented Apr 13, 2026

Uh oh!

jarjk commented Apr 22, 2026

Uh oh!

jarjk commented Apr 23, 2026

Uh oh!

Copilot AI commented Apr 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

jarjk commented Apr 23, 2026

Uh oh!

Copilot AI commented Apr 23, 2026

Uh oh!

jarjk Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

jarjk Apr 23, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Copilot AI commented Apr 11, 2026 •

edited

Loading

Copilot AI commented Apr 23, 2026 •

edited

Loading