Skip to content

chore: updates #88

Merged
beubax merged 9 commits into
mainfrom
chore/port-opencli-july-updates
Jul 15, 2026
Merged

chore: updates #88
beubax merged 9 commits into
mainfrom
chore/port-opencli-july-updates

Conversation

@rishabhraj36

Copy link
Copy Markdown
Collaborator

Description

Related issue:

Type of Change

  • 🐛 Bug fix
  • ✨ New feature
  • 🌐 New site adapter
  • 📝 Documentation
  • ♻️ Refactor
  • 🔧 CI / build / tooling

Checklist

  • I ran the checks relevant to this PR
  • I updated tests or docs if needed
  • I included output or screenshots when useful

Adapter Notes

  • Updated generated or lean docs when command discoverability changed
  • Used positional args for the command's primary subject unless a named flag is clearly better
  • Normalized expected adapter failures to CliError subclasses instead of raw Error

Screenshots / Output

@rishabhraj36 rishabhraj36 marked this pull request as ready for review July 15, 2026 08:18
@rishabhraj36

Copy link
Copy Markdown
Collaborator Author

🟠 Maintainer review suggested — low confidence

The pull request introduces a new `linkedin/company` command and significantly expands the `chatgpt/deep-research-result` command to include progress reporting. These changes introduce new user-facing capabilities and output schemas that are not reflected in the current documentation.

Limitations

  • One or more Gemini findings failed evidence validation.
  • Review context was truncated.

Source: gemini.

This review is advisory and does not block merging.

@rishabhraj36

Copy link
Copy Markdown
Collaborator Author

🔴 Documentation update likely missing — high confidence

The automated review found 3 likely missing documentation updates.

Findings

  1. README — A new command, `linkedin company`, has been added, allowing users to retrieve structured information about LinkedIn companies including name, industry, size, headquarters, and followers.
    • Changed: clis/linkedin/company.js
    • Evidence: @@ -0,0 +1,153 @@
    • Suggested update: README.md
    • Why: The `README.md` highlights several LinkedIn commands but does not include this new, significant command, making it undiscoverable for users browsing the main documentation.
  2. README — The `chatgpt deep-research-result` command now provides progress updates and additional details for ongoing deep research tasks, rather than only returning completed reports. It also returns early if the status is 'waiting_for_user' or 'needs_user_action'.
    • Changed: clis/chatgpt/deep-research-result.js
    • Evidence: description: 'Read a ChatGPT Deep Research report or progress from the conversation payload'
    • Suggested update: README.md
    • Why: The `README.md` mentions `chatgpt deep-research-result` but does not highlight its new capability to report progress or the additional output fields, which is a significant enhancement to its functionality.
  3. DOCS — Plugin installations now explicitly ignore lifecycle scripts (`preinstall`, `install`, `postinstall`) for enhanced security.
    • Changed: src/plugin.ts
    • Evidence: execFileSync('npm', ['install', '--omit=dev', '--ignore-scripts'], {
    • Suggested update: docs/plugins-and-skills.mdx
    • Why: The `docs/plugins-and-skills.mdx` document describes plugin installation but does not mention the security-related change that `npm install` now uses `--ignore-scripts`, which is a user-visible change in how plugins are handled.

Limitations

  • Some automated findings could not be verified against the pull request diff.

This review is advisory and does not block merging.

@beubax beubax merged commit 64ed887 into main Jul 15, 2026
25 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.

2 participants