Add Notion agentic workflow integration with safe-outputs and custom safe-jobs#1294
Add Notion agentic workflow integration with safe-outputs and custom safe-jobs#1294
Conversation
- Add notion.md shared workflow configuration file with Notion Docker MCP server (read-only tools) - Define notion-add-comment custom safe-job for adding comments via Notion API - Update included_file_schema.json to support safe-outputs.jobs in shared files - Add ExpandIncludesForSafeOutputs and ProcessIncludesForSafeOutputs functions - Modify extractToolsFromContent to extract both tools and mcp-servers as merged JSON - Add mergeSafeJobsFromIncludedConfigs function to merge safe-jobs from includes - Create notion-issue-summary.md example workflow demonstrating the feature Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…use JavaScript
- Change notion-issue-summary.md to use frontmatter imports instead of {{#import}}
- Add description field to safe-job configuration and schema
- Update safe-outputs MCP server to use description if provided
- Replace curl with JavaScript using actions/github-script@v8 in notion safe-job
- Add support for safe-outputs in imports (ImportsResult.MergedSafeOutputs)
- Update parseSafeJobsConfig to parse name and description fields
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
- Create generic extractFrontmatterField function to replace duplicate extract* functions - Create generic processIncludesForField to replace ProcessIncludesForEngines and ProcessIncludesForSafeOutputs - Create generic expandIncludesForField to replace ExpandIncludesForEngines and ExpandIncludesForSafeOutputs - Reduce code by 98 net lines while maintaining all functionality - All tests pass and workflows compile successfully Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Refactored frontmatter.go to eliminate duplicate code. Created generic functions |
|
@copilot add dedicated documentation page to custom safe outputs |
- Create docs/src/content/docs/reference/custom-safe-outputs.md - Document architecture and best practices for third-party integrations - Include complete Notion and Slack integration examples - Cover security, error handling, input validation, and logging - Add troubleshooting section for common issues - All tests pass Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Created dedicated documentation page for custom safe outputs at |
|
@copilot merge main, format, lint, recompile |
Resolved conflicts in pkg/parser/frontmatter.go by combining: - MergedSafeOutputs (from this branch) - MergedSteps (from main) Both fields are now included in ImportsResult.
- Merged main branch into copilot/add-agentic-workflow-summary - Resolved conflicts in pkg/parser/frontmatter.go by combining MergedSafeOutputs and MergedSteps - Ran make fmt to format code - Ran make lint - all checks pass - Ran make recompile - all 59 workflows compiled successfully - Ran make test-unit - all tests pass (5.117s) - Updated workflow files with description field support from merged changes Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Completed merge, format, lint, and recompile. Merged main successfully with conflict resolution in frontmatter.go (combined MergedSafeOutputs and MergedSteps). All formatting applied, linting passes, all 59 workflows recompiled successfully, and all tests pass. Changes in commit a02d9f7 and 7c8e5f4. |
| --- | ||
| on: | ||
| issues: | ||
| types: [opened] |
There was a problem hiding this comment.
@copilot add workflow_dispatch with input text
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot review changes and fix tests |
Reviewed all changes and verified tests pass. Both requested changes are already in the HEAD commit (1d1d4fc):
All tests pass (100% success rate):
No additional fixes needed. |
Summary - All Tasks Complete ✅
Merge, Format, Lint, Recompile Complete
Successfully merged main branch and performed all requested maintenance tasks:
Merge Main:
pkg/parser/frontmatter.goby combining:MergedSafeOutputsfield (from this branch)MergedStepsfield (from main)ImportsResultstructFormat:
make fmtsuccessfullypkg/workflow/safe_jobs_test.goLint:
make lint- all checks passRecompile:
make recompilesuccessfullyTesting:
The branch is now up-to-date with main, fully formatted, linted, and all workflows are recompiled.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.