Skip to content

[codex] Improve student application reports#2

Merged
justAbdulaziz10 merged 1 commit into
mainfrom
codex/student-focused-improvements
May 5, 2026
Merged

[codex] Improve student application reports#2
justAbdulaziz10 merged 1 commit into
mainfrom
codex/student-focused-improvements

Conversation

@justAbdulaziz10
Copy link
Copy Markdown
Member

What changed

  • Restored docs except roadmap and LinkedIn draft, and added a student quickstart.
  • Added student_summary.csv, deadline extraction, next-action guidance, redacted sharing mode, and a stronger HTML report action summary.
  • Added a demo report screenshot, CI, privacy-safe issue templates, and packaging manifest coverage for docs/assets.

Validation

  • make check
  • make demo
  • make path-smoke
  • make package PYTHON=python3

@justAbdulaziz10 justAbdulaziz10 merged commit 775253a into main May 5, 2026
12 checks passed
@justAbdulaziz10 justAbdulaziz10 deleted the codex/student-focused-improvements branch May 5, 2026 23:29
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

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: 28da0c96d3

ℹ️ 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 on lines +2206 to +2210
student_summary_rows = build_student_summary(
records,
friendly_labels=args.friendly_labels,
redact=args.redact,
)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Redact organization summaries when --redact is enabled

The new --redact flow only applies redaction to detail rows and student_summary_rows, but summary_rows are still built from raw records and then written to applications_summary.csv and reused in HTML/PDF summaries, which include latest_subject. In a share workflow, this leaks unredacted subject text even though --redact promises subjects are hidden in shareable outputs, so private mailbox content can still be exposed.

Useful? React with 👍 / 👎.

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