support multiple innies and multiple topics

[wolfdancer]

No description provided.

[Copilot]

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

Comments suppressed due to low confidence (1)

src/innieme/discord_bot.py:162

• Inside the loop iterating over 'topic.config.channels', this line reassigns 'channel' to the first element in the list, which results in only that channel being processed. Remove or modify this assignment to correctly use the loop variable.

channel = topic.config.channels[0]

[Copilot]

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

Comments suppressed due to low confidence (2)

src/innieme/document_processor.py:67

• The updated scan_and_vectorize method now requires a 'topic_name' parameter; please ensure all invocations are adjusted or consider providing a default if backward compatibility is needed.

async def scan_and_vectorize(self, topic_name:str) -> str:

src/innieme/discord_bot.py:185

• Confirm that topic.scan_and_vectorize() correctly passes the intended topic name, as the updated signature in DocumentProcessor requires it; consider explicitly providing the topic name to avoid runtime errors.

scanning_result = await topic.scan_and_vectorize()

[Copilot]

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

Comments suppressed due to low confidence (1)

src/innieme/discord_bot.py:185

• scan_and_vectorize now requires a 'topic_name' argument. Consider updating this call to pass topic.config.name or an appropriate identifier to ensure consistency with the updated signature.

scanning_result = await topic.scan_and_vectorize()