Skip to content

Refresh media comment action payload#2538

Merged
subzeroid merged 1 commit into
masterfrom
fix/comment-action-context
May 19, 2026
Merged

Refresh media comment action payload#2538
subzeroid merged 1 commit into
masterfrom
fix/comment-action-context

Conversation

@subzeroid

@subzeroid subzeroid commented May 19, 2026

Copy link
Copy Markdown
Owner

Summary

  • refresh media_comment write payload to include current action context fields used by feed write actions
  • add regression coverage for comment creation payload and reply handling
  • fix live account helper URL building so workflow account pools are not accidentally pinned to one stale account
  • fix the live comment test to upload its own media, call media_comment, read the created comment back, and cleanup via comment_bulk_delete / media_delete
  • add a dedicated comment target to the live account workflow
  • document that comment creation remains a write action subject to Instagram spam/trust checks

Refs #2097.

Tests

  • .venv/bin/python -m pytest tests/regression/test_comments.py -q
  • .venv/bin/python -m pytest tests/regression/test_helpers.py tests/regression/test_comments.py -q
  • .venv/bin/python -m pytest tests/regression/test_comments.py tests/regression/test_media.py::MediaActionPayloadRegressionTestCase tests/regression/test_hardening.py::HardeningRegressionTestCase::test_send_private_request_promotes_404_not_found_body_to_challenge -q
  • .venv/bin/python -m pytest -sv tests/regression/test_extractors.py::ExtractorsRegressionTestCase tests/regression/test_public.py::PublicRegressionTestCase tests/regression/test_public.py::PrivateGraphQLRequestRegressionTestCase tests/regression/test_direct.py::DirectMixinRegressionTestCase tests/regression/test_challenge.py::ChallengeRegressionTestCase tests/regression/test_comments.py::CommentRepliesRegressionTestCase tests/regression/test_current_app_profile.py::CurrentAppProfileRegressionTestCase tests/regression/test_auth_story.py::AuthAndStoryRegressionTestCase tests/regression/test_download.py::DownloadRegressionTestCase tests/regression/test_notes.py::NoteMixinRegressionTestCase tests/regression/test_location.py::LocationMixinRegressionTestCase tests/regression/test_media.py::UserMediasGraphQLRegressionTestCase tests/regression/test_user.py::UserMixinRegressionTestCase tests/regression/test_timeline.py::TimelineRegressionTestCase tests/regression/test_story_configure.py::StoryConfigureRegressionTestCase tests/regression/test_video_metadata.py::VideoMetadataRegressionTestCase tests/regression/test_upload.py::UploadRegressionTestCase
  • .venv/bin/python -m ruff check instagrapi tests
  • .venv/bin/python -m ruff format --check instagrapi tests
  • .venv/bin/python -m mkdocs build --strict
  • .venv/bin/python -m build

Live verification

@subzeroid subzeroid force-pushed the fix/comment-action-context branch from 7066515 to 67c910e Compare May 19, 2026 20:42
@subzeroid subzeroid force-pushed the fix/comment-action-context branch from 67c910e to 3e25e53 Compare May 19, 2026 20:47
@subzeroid subzeroid merged commit 90d1d6c into master May 19, 2026
12 checks passed
@subzeroid subzeroid deleted the fix/comment-action-context branch May 19, 2026 20:52
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