Skip to content

Count the example verify step in progress#94

Merged
madebygps merged 1 commit into
mainfrom
madebygps/update-verify-progress-count-from-18-to
May 28, 2026
Merged

Count the example verify step in progress#94
madebygps merged 1 commit into
mainfrom
madebygps/update-verify-progress-count-from-18-to

Conversation

@madebygps
Copy link
Copy Markdown
Collaborator

This makes learner-facing progress match what the CLI already records after verify 0 CTF{example}. The example verification now counts toward the visible total, so learners see 1/19 after the bootstrap step without changing certificate or token completion requirements.

Fixes: #87

What changed

  • split verify progress counting from certificate/export completion counting
  • show progress out of 19 total checks, including the example verification
  • keep export and token behavior tied to the 18 real challenges
  • update learner docs and the CTF test script to reflect the new 1/19 and 19/19 progress output

Notes

  • This intentionally preserves the existing export threshold and token payload semantics at 18 real challenges.
  • The test script now checks that the example step advances progress immediately.

Validation

  • python -m compileall verify\src
  • targeted verify command checks for 1/19, 19/19, and Current progress: 2/18 export messaging

Update learner-facing progress to count the example verification as part of the 19 total checks while keeping export and token completion semantics tied to the 18 real challenges.

Refresh the CTF test script and docs to match the new 1/19 and 19/19 progress behavior.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@madebygps
Copy link
Copy Markdown
Collaborator Author

I manually and locally tested this change and it works great.

@madebygps madebygps merged commit f3e3efe into main May 28, 2026
4 checks passed
@madebygps madebygps deleted the madebygps/update-verify-progress-count-from-18-to branch May 28, 2026 13:42
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.

Update verify progress count from 18 to 19

1 participant