Skip to content

Changes to comment structure#490

Merged
JimDaly merged 3 commits intomainfrom
u/jdaly/comment-fixes
Jul 23, 2025
Merged

Changes to comment structure#490
JimDaly merged 3 commits intomainfrom
u/jdaly/comment-fixes

Conversation

@JimDaly
Copy link
Copy Markdown
Member

@JimDaly JimDaly commented Jul 23, 2025

Changes based on learning of how docFx and type2docfx expects '@' tags to be used in code comments.

@JimDaly JimDaly requested review from Copilot and rido-min July 23, 2025 21:45
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR restructures comment documentation throughout the codebase to align with docFx and type2docfx formatting expectations by properly organizing @ tags and remarks sections.

  • Moves descriptive content from initial comment text to @remarks sections
  • Reorganizes parameter documentation (@param) to appear before @remarks sections
  • Standardizes the placement and formatting of JSDoc tags across multiple packages

Reviewed Changes

Copilot reviewed 48 out of 49 changed files in this pull request and generated 11 comments.

Show a summary per file
File Description
packages/agents-hosting/src/turnContext.ts Restructures JSDoc comments for TurnContext methods to move descriptions to @remarks
packages/agents-hosting/src/storage/ Updates storage-related classes with proper @remarks formatting
packages/agents-hosting/src/state/ Reorganizes state management documentation structure
packages/agents-hosting/src/app/ Standardizes application framework documentation formatting
packages/agents-hosting/src/auth/authConfiguration.ts Moves parameter documentation before @remarks sections
packages/agents-hosting-dialogs/src/ Updates dialog framework documentation structure
packages/agents-hosting-storage-blob/src/ Restructures blob storage documentation
packages/agents-hosting-express/src/ Updates Express integration documentation
packages/agents-copilotstudio-client/src/ Reorganizes Copilot Studio client documentation
packages/agents-activity/src/ Updates activity framework documentation structure
Comments suppressed due to low confidence (1)

Comment thread packages/agents-hosting/src/storage/storage.ts
Comment thread packages/agents-hosting/src/storage/storage.ts
Comment thread packages/agents-hosting/src/storage/storage.ts
Comment thread packages/agents-hosting/src/storage/storage.ts
Comment thread packages/agents-hosting/src/storage/fileStorage.ts
Comment thread packages/agents-hosting/src/cards/adaptiveCard.ts
Comment thread packages/agents-hosting/src/auth/authConfiguration.ts
Comment thread packages/agents-hosting-dialogs/src/recognizerResult.ts
Comment thread packages/agents-hosting-dialogs/src/choices/findValues.ts
Comment thread packages/agents-hosting-dialogs/src/dialogInstance.ts
@JimDaly JimDaly enabled auto-merge (squash) July 23, 2025 22:01
@JimDaly JimDaly merged commit 444d99f into main Jul 23, 2025
7 checks passed
@JimDaly JimDaly deleted the u/jdaly/comment-fixes branch July 23, 2025 23:13
@JimDaly JimDaly self-assigned this Aug 11, 2025
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.

3 participants