-
Notifications
You must be signed in to change notification settings - Fork 0
fix!: migrate examples and tests to connector-based tool naming #51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Replace legacy unified API naming conventions (hris_*, ats_*, crm_*) with connector-based naming (hibob_*, bamboohr_*, workday_*) across all example files. This aligns the examples with the updated StackOne API architecture where tools are named after the specific connector/provider rather than abstract unified API categories. Affected examples: - crewai_integration.py - file_uploads.py - index.py - langchain_integration.py - meta_tools_example.py - openai_integration.py - stackone_account_ids.py
Update all test files to use connector-based naming conventions (hibob_*, bamboohr_*, workday_*) instead of legacy unified API naming (hris_*, ats_*, crm_*). This ensures tests remain consistent with the updated examples and reflect the actual tool naming used by StackOne connectors. Affected tests: - test_feedback.py - test_meta_tools.py - test_tfidf_index.py - test_toolset.py
Update the docstring usage example in langgraph.py to use connector-based naming (hibob_*) and the current fetch_tools API instead of the deprecated get_tools method with unified API naming.
There was a problem hiding this 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 migrates all examples and tests from legacy unified API naming conventions (e.g., hris_*, ats_*, crm_*) to connector-specific naming (e.g., hibob_*, bamboohr_*, workday_*), aligning with StackOne's architectural shift from abstract API categories to provider-specific tool naming.
Key Changes:
- Updated 7 example files to use connector-based tool name patterns
- Updated 4 test files with connector-specific assertions and fixtures
- Revised documentation strings to reference connectors instead of unified API categories
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
tests/test_toolset.py |
Updated filter and pattern matching tests to use connector names (hibob, bamboohr, workday) instead of API categories |
tests/test_tfidf_index.py |
Updated tokenization and search tests with connector-based tool names |
tests/test_meta_tools.py |
Updated fixtures to generate HiBob and BambooHR tools instead of HRIS and ATS tools |
tests/test_feedback.py |
Updated feedback submission test to reference hibob_list_employees |
stackone_ai/integrations/langgraph.py |
Updated docstring example to show connector-based tool filtering |
examples/stackone_account_ids.py |
Updated to demonstrate HiBob tool filtering and retrieval |
examples/openai_integration.py |
Updated to use HiBob-specific tools in OpenAI integration example |
examples/meta_tools_example.py |
Updated all examples to use HiBob tools instead of HRIS tools |
examples/langchain_integration.py |
Updated to filter HiBob tools in LangChain integration |
examples/index.py |
Updated quickstart example and removed "Unified" from API description |
examples/file_uploads.py |
Updated file upload example to reference HiBob document upload tool |
examples/crewai_integration.py |
Updated to filter HiBob tools in CrewAI integration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No issues found across 12 files
examples/file_uploads.py
Outdated
| # Resume content | ||
| This is a sample resume content that will be uploaded using the `hris_upload_employee_document` tool. | ||
| This is a sample resume content that will be uploaded using the `hibob_upload_employee_document` tool. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For all the examples, please use an actual existing tool from one of our Falcon connector
Address PR review feedback requesting actual existing Falcon connector tools. Migrate all examples from hibob_* to bamboohr_* tools which are confirmed to exist in production Falcon connectors. Changes include: - file_uploads.py: bamboohr_upload_employee_document - index.py: bamboohr_list_employees - openai_integration.py: bamboohr_get_employee, bamboohr_*_employments - All other examples: bamboohr_* action filters This ensures examples demonstrate tools that actually exist in the StackOne platform.
…onnector-based-naming
glebedel
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Summary
Migrates all examples and tests from legacy unified API naming conventions (
hris_*,ats_*,crm_*) to connector-based naming (hibob_*,bamboohr_*,workday_*).What Changed
Why
The StackOne API architecture has moved from abstract unified API categories to connector-specific naming. This change:
Breaking Change
This is marked as a breaking change because users following the old examples will need to update their tool name patterns when filtering tools (e.g.,
actions=["hibob_*"]instead ofactions=["hris_*"]).Summary by cubic
Migrates examples, tests, and LangGraph docstring from unified API prefixes (hris_, ats_, crm_) to connector-based prefixes (hibob_, bamboohr_, workday_). Aligns usage with the current StackOne API and breaks code that still uses legacy names.
Refactors
Migration
Written for commit e963b5a. Summary will update automatically on new commits.