Fix NLTK data PermissionError in Docker

[patrick91]

Summary

• Add NLTK_DATA env var pointing to /home/app/nltk_data with correct ownership for the app user
• Same root cause as Add cache for HF #4574: su -p preserves HOME=/root, so NLTK defaults to /root/nltk_data which the app user cannot write to
• docs https://www.nltk.org/data.html

Test plan

• Deploy and trigger similar_talks analysis that downloads NLTK stopwords
• Verify no PermissionError for /root/nltk_data in logs

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
pycon	Building	Preview	Feb 9, 2026 8:21pm

[claude]

Dockerfile change adds NLTK_DATA environment variable and creates the directory with proper ownership for the app user, fixing permission errors when NLTK downloads stopwords data.

The fix correctly addresses the root cause: su -p preserves HOME=/root, so without the explicit NLTK_DATA env var, NLTK would default to /root/nltk_data which the app user cannot write to. The solution mirrors the existing HF_HOME pattern.

No issues found.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.52%. Comparing base (32fc459) to head (4c63602).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4575   +/-   ##
=======================================
  Coverage   92.52%   92.52%           
=======================================
  Files         357      357           
  Lines       10690    10690           
  Branches      812      812           
=======================================
  Hits         9891     9891           
  Misses        687      687           
  Partials      112      112           

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.