Skip to content

Tighten StreetPass documentation and private logging - #1

Merged
theos2node merged 1 commit into
mainfrom
agent/professional-review
Jul 28, 2026
Merged

Tighten StreetPass documentation and private logging#1
theos2node merged 1 commit into
mainfrom
agent/professional-review

Conversation

@theos2node

Copy link
Copy Markdown
Owner

Summary

  • replace third-person owner claims with a repeatable measured test baseline
  • keep Bluetooth profile data and peripheral identifiers private in unified logs
  • normalize the changelog around the existing open-source release

Verification

  • XcodeGen 2.45.4 produced no project changes
  • configuration and helper scripts validated
  • StreetPass unit tests passed
  • StreetPass UI tests passed on an iPhone 17 simulator
  • full Git history and working tree scanned for secrets

@theos2node
theos2node marked this pull request as ready for review July 28, 2026 05:50

@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: 1f39d167ce

ℹ️ 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".

- a comparable baseline without StreetPass active.

The project owner’s physical stress testing found no material battery effect. Treat that as a strong encouraging observation, while continuing to measure new hardware and iOS releases.
Repeated physical stress tests found no material battery change against the test baseline. Continue to measure after hardware, radio, or iOS changes.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Regenerate the checked-in playbook PDF

The Markdown now replaces the owner-specific battery claim, but APPLE_BLE_BACKGROUND_PLAYBOOK.pdf remains byte-for-byte unchanged from the parent commit and still embeds the old “project owner's physical stress testing” wording. Since this repository includes Scripts/render_markdown_pdf.py for generating that PDF, regenerate and commit it so users of the PDF receive the same tightened claim as readers of the Markdown.

Useful? React with 👍 / 👎.

@theos2node
theos2node merged commit 30dd7b2 into main Jul 28, 2026
1 check passed
@theos2node
theos2node deleted the agent/professional-review branch July 28, 2026 05:56
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