Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix spelling, grammar, and minor formatting issues in running agents documentation #1128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 18, 2025

Conversation

mshsheikh
Copy link
Contributor

@mshsheikh mshsheikh commented Jul 15, 2025

Problem:
The documentation for agent execution contains several minor but distracting issues:

  • Verb tense mismatch ("outputs produces" → "outputs produced")
  • Optional inconsistencies in code examples (haiku punctuation, undefined thread_id)

Changes:

  1. Grammar/Spelling Fixes
  • Changed "outputs produces""outputs produced" (corrected past-tense verb agreement)
  1. Optional Code Improvements
  • Removed period from haiku example for stylistic consistency python # Infinite loop's dance. → # Infinite loop's dance
  • Added explicit thread_id definition in manual conversation example python thread_id = "thread_123" # Example thread ID

…documentation

**Problem**:
The documentation for agent execution contains several minor but distracting issues:
- Incorrect pluralization ("OpenAI Responses API" → "OpenAI Response API")
- Verb tense mismatch ("outputs produces" → "outputs produced")
- Optional inconsistencies in code examples (haiku punctuation, undefined `thread_id`)

**Changes**:
1. **Grammar/Spelling Fixes**
- Changed `"OpenAI Responses API"` → `"OpenAI Response API"`  
     (singular "Response" aligns with the singular API concept)
- Changed `"outputs produces"` → `"outputs produced"`  
     (corrected past-tense verb agreement)

2. **Optional Code Improvements**
- Removed period from haiku example for stylistic consistency  
     ```python
     # Infinite loop's dance.  →  # Infinite loop's dance
     ```
- Added explicit `thread_id` definition in manual conversation example  
     ```python
     thread_id = "thread_123"  # Example thread ID
     ```

**Benefits**:
- Improves readability and professionalism of documentation.
- Fixes technical inaccuracies in API references and grammar.
- Ensures code examples are self-contained and executable without undefined variables.
- Maintains stylistic consistency with project standards.

All changes preserve the original technical accuracy and intent of the documentation while addressing surface-level issues that could confuse readers or disrupt workflow.
@seratch seratch added the documentation Improvements or additions to documentation label Jul 15, 2025
```

Read more in the [results guide](results.md).

## The agent loop

When you use the run method in `Runner`, you pass in a starting agent and input. The input can either be a string (which is considered a user message), or a list of input items, which are the items in the OpenAI Responses API.
Copy link
Member

Choose a reason for hiding this comment

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

Responses API is the correct name; please revert this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks! I’ve reverted the terminology back to “OpenAI Responses API” throughout the section.
Please let me know if there’s anything else that needs adjustment.

This update cleans up minor issues in the `running_agents.md` documentation:

- **Grammar Fix:** Corrects “outputs produces” → “outputs produced” for proper past-tense agreement.
- **Code Example Improvements:**

- Removes the trailing period from the haiku comment (`# Infinite loop's dance.` → `# Infinite loop's dance`).
- Defines `thread_id` explicitly (`thread_id = "thread_123"`) so the manual conversation snippet is self-contained and runnable.

These changes enhance readability and ensure all examples are clear and immediately executable without altering any core technical content.
@mshsheikh mshsheikh requested a review from seratch July 16, 2025 00:13
@mshsheikh
Copy link
Contributor Author

Waiting for the experts review here from @seratch and @rm-openai 🎓

@seratch seratch merged commit bc32b99 into openai:main Jul 18, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants