Skip to content

Add Doordash app#68

Open
HuanCC666 wants to merge 3 commits into
mainfrom
add_doordash_app
Open

Add Doordash app#68
HuanCC666 wants to merge 3 commits into
mainfrom
add_doordash_app

Conversation

@HuanCC666

@HuanCC666 HuanCC666 commented Jan 12, 2026

Copy link
Copy Markdown
Collaborator

Description

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactoring (code restructuring without changing behavior)
  • Documentation update
  • Test coverage improvement
  • Performance improvement
  • Chore (dependency updates, CI/CD, tooling)

Related Issues

Closes #

Related Design Documents

Component

  • Core (All files in the root pas, e.g. pas/environment.py)
  • Agents (All files in pas/agents/)
  • Scenarios (All files in pas/scenarios/)
  • Scenario Generator (All files in pas/scenario_generator/)
  • Apps (All files in pas/apps/)
  • Documentation (Documentation files e.g., docs/, mkdocs.yml, README.md)
  • Testing Infrastructure
  • Other

Changes Made

  • Added Doordash app and tests
    -Added doordash doc

Breaking Changes

  • This PR introduces breaking changes
  • Migration guide included (if applicable)

Details:

Testing

Test Coverage

  • Unit tests added/updated
  • Integration tests added/updated
  • All tests pass locally (make test or uv run pytest)
  • Test coverage maintained or improved

Manual Testing

Test steps:
1.
2.
3.

Test environment:

  • OS:
  • Python version:
  • Relevant models used (if applicable):

Code Quality

  • Pre-commit hooks pass (see output below)
  • No dependency issues (uv run deptry pas)
  • Code follows project style guidelines (f-strings, Google docstrings, etc.)

Pre-commit output:

image
# Paste output here

Documentation

  • Docstrings added/updated (Google style)
  • mkdocs documentation updated (if applicable)
  • Design documents updated (if implementation diverged from plan)
  • README updated (if user-facing changes)

Checklist

  • My code follows the project's coding standards
  • I have performed a self-review of my code
  • I have commented complex/non-obvious code sections
  • My changes generate no new warnings
  • I have added tests that prove my fix/feature works
  • New and existing tests pass locally
  • Any dependent changes have been merged and published

Screenshots/Logs (if applicable)

@deepakn97 deepakn97 self-requested a review January 12, 2026 22:17

@deepakn97 deepakn97 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please change the name of the app to food_delivery instead of Doordash. We cannot officially name our app Doordash. Same with the venmo app. Name it payments

@HuanCC666

Copy link
Copy Markdown
Collaborator Author

ok

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