Skip to content

Conversation

@AaronDDM
Copy link
Collaborator

@AaronDDM AaronDDM commented Sep 10, 2025

What did you do?

  • Added an esm only example
  • Updated deps for our examples

License

I confirm that this contribution is made under the terms of the MIT license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

@codecov
Copy link

codecov bot commented Sep 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.87%. Comparing base (1a3e15a) to head (5bbc1d0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #664   +/-   ##
=======================================
  Coverage   94.87%   94.87%           
=======================================
  Files          35       35           
  Lines         781      781           
  Branches       66       66           
=======================================
  Hits          741      741           
  Misses         33       33           
  Partials        7        7           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@playerzero-ai
Copy link

playerzero-ai bot commented Sep 10, 2025

Pull Request Summary

The pull request primarily focuses on code style and formatting improvements across various example files, with no changes to product functionality or user experience. Here are the key changes:

  • General Formatting and Style:

    • Converted double quotes to single quotes, added trailing commas, and adjusted line breaks and indentation for better readability.
    • Reformatted console logs and error messages for clarity.
    • Ensured consistent use of arrow function syntax and added missing newlines at the end of files.
  • Logging Enhancements:

    • Improved logging in notetaker.ts for better diagnostics and visibility.
    • Introduced a new logger utility (logger.mjs) for consistent and human-friendly log formatting with ANSI color codes and secret masking.
  • New ESM Example:

    • Added an ESM-only example (indes.mjs) demonstrating pure ESM import usage, environment variable handling, and a minimal message-listing flow.
  • Environment Variable Handling:

    • Added dotenv configuration in send-attachments-cli.ts to load environment variables, although its placement may not affect the current logic due to ordering.
  • Code Readability and Maintenance:

    • Reformatted function signatures and object literals across multiple lines for improved readability.
    • Made minor syntactic fixes and whitespace adjustments to satisfy linting rules and maintain consistency.

Overall, the pull request enhances code readability and maintainability without altering the existing functionality or user experience.

Files Changed

File Name Summary
examples/calendars/event_with_notetaker.ts Purely stylistic: quotes, trailing commas, line breaks, minor syntactic fixes. No behavioral/API changes.
examples/messages/examples/stream-attachments.ts Formatting and lint cleanups (imports, trailing commas, line-wrapping). No functional changes.
examples/folders/folders.ts Style-only edits (whitespace, multi-line logs/arrow funcs). API calls and behavior unchanged.
examples/messages/messages.ts Broad formatting/log reflow and trailing commas; no change to API usage or logic.
examples/messages/cli-interface.ts CLI formatting/refactors and log reflow; commands, options and behavior unchanged.
examples/notetakers/notetaker.ts Added more informative logging and more robust media-date handling (accepts snake_case/camelCase). No API behavior changes.
examples/utils/logger.mjs New file: pretty console logger with timestamp, level colors, symbols, secret masking, and logger methods.
examples/messages/send-attachments-cli.ts Stylistic changes and final newline. dotenv.config(...) added but placed after env checks (likely no effect). Core behavior unchanged.
examples/messages/examples/file-path-attachments.ts Formatting/whitespace cleanups and trailing commas; no runtime or API changes.
examples/esm-only/indes.mjs New ESM-only example: loads examples/.env, validates creds, lists messages, masks API key, exits non-zero on error.
examples/messages/utils/attachment-file-manager.ts Readability/formatting-only edits; logic (file checks, size limits, builders) unchanged.
examples/messages/examples/flexible-attachments.ts Style-only reflow and trailing commas; attachment logic and timeouts unchanged.
examples/messages/examples/buffer-attachments.ts Formatting/spacing fixes; no change to sending/attachment behavior.
examples/edge-environment/src/worker.ts Minor formatting and line-wrapping; request/CORS handling and behavior unchanged.
examples/messages/examples/index.ts Type and signature reflow for readability; no change to function contracts or behavior.
examples/messages/examples/string-attachments.ts Formatting and small syntax normalizations; attachment selection and send logic unchanged.
examples/messages/examples/file-path-attachments.ts (Duplicate entry) Formatting/whitespace cleanups and trailing commas; no runtime or API changes.
examples/messages/examples/stream-attachments.ts (Duplicate entry) Formatting and lint cleanups (imports, trailing commas, line-wrapping). No functional changes.
examples/messages/send-attachments-cli.ts (Duplicate entry) Stylistic changes; dotenv loading added after env checks (likely ineffective). No behavioral change.
examples/messages/messages.ts (Duplicate entry) Broad formatting/log reflow and trailing commas; no change to API usage or logic.

View more in PlayerZero
updated: Sep 10 @ 01:28 PM UTC

@AaronDDM AaronDDM merged commit bc4bbb0 into main Sep 10, 2025
7 checks 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.

4 participants