tweak prompt and hoist mod tools

[zzstoatzz]

No description provided.

[Copilot]

Pull Request Overview

Refactors GitHub integration into its own module, replaces hard-coded slice length with a named constant, and adds module-exploration tools to the agent.

• Extract GitHub token retrieval and Pydantic models into slackbot.github
• Introduce TOKEN_LIMIT constant and replace the literal 1000 in slice_tokens
• Register display_callable_signature and explore_module_offerings as agent tools and refine the research prompt

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
examples/slackbot/src/slackbot/search.py	Hoist GitHub logic, import _get_token & GitHubIssue, add TOKEN_LIMIT constant
examples/slackbot/src/slackbot/research_agent.py	Enhance prompt text with clarifications and an empirical-syntax rule
examples/slackbot/src/slackbot/github.py	New module defining _get_token and Pydantic models for GitHub entities
examples/slackbot/src/slackbot/core.py	Import and register display_callable_signature & explore_module_offerings tools

Comments suppressed due to low confidence (5)

examples/slackbot/src/slackbot/research_agent.py:63

• [nitpick] This line isn’t numbered like the surrounding steps. Prefix it with a step number or bullet to maintain consistent formatting in the prompt.

Do not use any Prefect syntax you have not gathered empirically.

examples/slackbot/src/slackbot/core.py:25

• Neither display_callable_signature nor explore_module_offerings are defined in slackbot.search. This import will fail at runtime; you should import them from the modules where they’re actually defined (e.g., pretty_mod).

display_callable_signature,

examples/slackbot/src/slackbot/github.py:63

• default_factory should create a list[GitHubLabel], not a single GitHubLabel instance. Use default_factory=list to produce an empty list.

labels: list[GitHubLabel] = Field(default_factory=GitHubLabel)

examples/slackbot/src/slackbot/github.py:62

• The original validator that coerced None to "" was removed. If the API returns null for body, Pydantic will error on a str field. Consider re-adding a validator or making the field Optional[str] with a default of "".

body: str = Field(default="")

examples/slackbot/src/slackbot/search.py:166

• [nitpick] Defining TOKEN_LIMIT inside the function redeclares it on every call. Move it to the module level (or a config) so it’s only defined once and can be shared.

TOKEN_LIMIT = 1500