Skip to content

feat(inter-tool-launch): implement Inter-Tool Launch Context (Issue #161)#523

Draft
Copilot wants to merge 5 commits intorelease/1.2.2from
copilot/create-implementation-plan-for-161
Draft

feat(inter-tool-launch): implement Inter-Tool Launch Context (Issue #161)#523
Copilot wants to merge 5 commits intorelease/1.2.2from
copilot/create-implementation-plan-for-161

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 11, 2026


  • Create .github/plans/plan-inter-tool-launch-context.md following the repo plan template
  • Part 1 — pptb.config.json schema: pptbConfig.d.ts, validate.js, pptb-validate.js
  • Part 2 — IPC Channels: LAUNCH_WITH_CONTEXT, RETURN_INVOCATION_DATA
  • Part 3 — Main process: pendingInvocations map, launchToolWithContext, resolveInvocation, null-resolve on premature close
  • Part 4 — Tool Preload Bridge: toolboxAPI.invocation namespace (getLaunchContext, returnData, launchTool)
  • Part 5 — Renderer: LaunchToolOptions extended, launchToolWithContext routing, preload.ts exposure
  • Part 6 — Public types: InvocationAPI in packages/toolboxAPI.d.ts, launchToolWithContext in api.ts
  • Part 7 — Documentation: packages/README.md inter-tool section + API reference
  • Execution log, code reviewer fixes (empty-object guard, blank line, .gitignore), security scan results, files-changed table, validation steps, risks all documented
  • Renamed PptbConfigPPTBConfig and validatePptbConfigvalidatePPTBConfig across packages/pptbConfig.d.ts, packages/lib/validate.js, packages/bin/pptb-validate.js, and the plan file for consistent PPTB casing

Copilot AI and others added 3 commits April 11, 2026 03:37
…semver enforcement

Agent-Logs-Url: https://github.com/PowerPlatformToolBox/desktop-app/sessions/f5d8d8b6-bbf6-473f-a251-979e002c64a3

Co-authored-by: Power-Maverick <36135520+Power-Maverick@users.noreply.github.com>
…ng all 7 implemented parts

Agent-Logs-Url: https://github.com/PowerPlatformToolBox/desktop-app/sessions/16f5bfb7-7f3f-4b6c-9829-843603515c18

Co-authored-by: Power-Maverick <36135520+Power-Maverick@users.noreply.github.com>
@Power-Maverick
Copy link
Copy Markdown
Contributor

@copilot rename PptbConfig to PPTBConfig, validatePptbConfig to validatePPTBConfig. Also update the copilot instructions plus agents for word "PPTB"; it should either be "PPTB" or "pptb" cannot be mix of upper and lower casing

…Config for consistent casing

Agent-Logs-Url: https://github.com/PowerPlatformToolBox/desktop-app/sessions/f894c5cf-a937-472f-a7ee-823d77dc6687

Co-authored-by: Power-Maverick <36135520+Power-Maverick@users.noreply.github.com>
Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 13, 2026

@copilot rename PptbConfig to PPTBConfig, validatePptbConfig to validatePPTBConfig. Also update the copilot instructions plus agents for word "PPTB"; it should either be "PPT...

Done in cfd03dc. Renamed PptbConfigPPTBConfig and validatePptbConfigvalidatePPTBConfig across packages/pptbConfig.d.ts, packages/lib/validate.js, packages/bin/pptb-validate.js, and the plan file.

Regarding copilot instructions and agents: no mixed-casing of "PPTB" was found in .github/copilot-instructions.md or any of the agent files — they either use the full uppercase PPTB or the lowercase pptb (as a filename prefix like pptb.config.json or pptb-validate). Let me know if there's a specific file or agent where you spotted the issue.

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