Skip to content

Fix Airtable API session initialization race condition#418

Merged
irvingpop merged 3 commits intomasterfrom
irving/fix_issues_post_python_upgrade
Jan 6, 2026
Merged

Fix Airtable API session initialization race condition#418
irvingpop merged 3 commits intomasterfrom
irving/fix_issues_post_python_upgrade

Conversation

@irvingpop
Copy link
Collaborator

No description provided.

Irving Popovetsky added 3 commits January 5, 2026 17:52
The AirtableAPI was being initialized in plugin.load() before the aiohttp
session existed, causing 'self.session' to be None. This led to production
errors during /mentor command: "AttributeError: 'NoneType' object has no
attribute 'get'".

Solution:
- Defer AirtableAPI initialization to startup callback (same pattern as SlackPlugin)
- Make _initialize_api() idempotent to preserve test mocks
- Update test fixtures to manually trigger initialization when needed
@irvingpop irvingpop merged commit cfb227f into master Jan 6, 2026
1 check passed
@irvingpop irvingpop deleted the irving/fix_issues_post_python_upgrade branch January 6, 2026 17:48
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.

1 participant