Skip to content

Jt playwrite#388

Open
JackTelford wants to merge 4 commits intoCrestApps:mainfrom
JackTelford:jt-Playwrite
Open

Jt playwrite#388
JackTelford wants to merge 4 commits intoCrestApps:mainfrom
JackTelford:jt-Playwrite

Conversation

@JackTelford
Copy link
Copy Markdown
Contributor

This pull request introduces the new CrestApps.OrchardCore.AI.Playwright module for browser automation, updates documentation to describe its features, and adds the Microsoft.Playwright package dependency. It also provides detailed changelog entries for the new module and removes unnecessary peer: true fields from the package-lock.json file.

AI Playwright Browser Automation module:

  • Added CrestApps.OrchardCore.AI.Playwright project to the solution and included Microsoft.Playwright as a dependency for browser automation workflows. [1] [2]
  • Introduced and documented the Playwright module, describing persistent browser sessions, live page observation tools, and content-aware editing for Orchard Core admin workflows in playwright.md and updated the main AI documentation index. [1] [2]
  • Updated the changelog to highlight Playwright improvements such as persistent sessions, inactivity cleanup, interactive tools, smarter editing flows, and clearer error reporting.

Documentation and codebase maintenance:

@github-actions
Copy link
Copy Markdown

This pull request has merge conflicts. Please resolve those before requesting a review.

{
if (GetSession(chatSessionId, ownerId) is PlaywrightSession session)
{
_logger.LogInformation("Stopping Playwright session '{ChatSessionId}'.", chatSessionId);

Check failure

Code scanning / CodeQL

Log entries created from user input High

This log entry depends on a
user-provided value
.

if (_sessions.TryRemove(chatSessionId, out _))
{
_logger.LogInformation("Closing Playwright session '{ChatSessionId}'.", chatSessionId);

Check failure

Code scanning / CodeQL

Log entries created from user input High

This log entry depends on a
user-provided value
.
@github-actions
Copy link
Copy Markdown

This pull request has merge conflicts. Please resolve those before requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant