Skip to content

Add debug logging for createStandup#7

Merged
oohwooh merged 1 commit into
codeday:mainfrom
dotcomaki:debug/create-standup-logging
Jun 25, 2026
Merged

Add debug logging for createStandup#7
oohwooh merged 1 commit into
codeday:mainfrom
dotcomaki:debug/create-standup-logging

Conversation

@dotcomaki

Copy link
Copy Markdown
Contributor

Trace event load, per-project Slack invite, standup creation payload/response, and emit a final created/skipped/failed summary to make standup creation issues easier to diagnose.

Trace event load, per-project Slack invite, standup creation payload/response, and emit a final created/skipped/failed summary to make standup creation issues easier to diagnose.
@augmentcode

augmentcode Bot commented Jun 22, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: Adds richer debug logging to the createStandup task to make standup-creation failures easier to diagnose.

Changes:

  • Logs event load details and the number of candidate projects
  • Adds per-project tracing for Slack bot invites and student eligibility
  • Logs the Standup & Prosper create payload/response and persists standupId when present
  • Emits a final created/skipped/failed summary for the run

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode 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.

Review completed. 1 suggestion posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

}

DEBUG(project.students);
DEBUG(`Project ${project.id} students: %O`, project.students);

@augmentcode augmentcode Bot Jun 22, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

project.students includes student email (see SlackStudentInfo / slackEventInfoSelect), so logging it with %O will emit PII whenever debug logging is enabled; consider restricting/redacting logged fields (e.g., counts/IDs only).

Severity: medium

Other Locations
  • src/activities/tasks/createStandup.ts:135
  • src/activities/tasks/createStandup.ts:140

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

@oohwooh oohwooh left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@oohwooh oohwooh merged commit 869f53b into codeday:main Jun 25, 2026
1 check passed
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.

2 participants