Skip to content

Fix Python 3.10 syntax incompatibilities breaking the build job#12

Merged
fivepanelhat merged 6 commits into
mainfrom
copilot/fix-failing-actions-job
Jun 28, 2026
Merged

Fix Python 3.10 syntax incompatibilities breaking the build job#12
fivepanelhat merged 6 commits into
mainfrom
copilot/fix-failing-actions-job

Conversation

Copilot AI commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

The build job was failing in the flake8 syntax gate under GitHub Actions' Python 3.10 environment. The failure came from invalid multiline f-string patterns and a placeholder orchestrator.py implementation that could not be parsed.

  • Python 3.10 syntax cleanup

    • Rewrote multiline f-strings into parser-safe string construction in the files flagged by CI:
      • agents.py
      • langgraph/embeddings.py
      • langgraph/ingestion.py
      • models.py
      • src/attestation_agent.py
      • src/fleet_manager.py
  • Demo orchestrator repair

    • Replaced the invalid placeholder implementation in orchestrator.py with a minimal, lint-safe demo orchestrator.
    • Kept the file scoped to the existing demo path: security check, telemetry wrapper, and reusable IntakeAgent wiring.
  • Stress script payload formatting

    • Replaced hand-built interpolated JSON in tests/stress/mqtt_flood.py with json.dumps(...) to avoid syntax fragility and preserve the emitted payload structure.

Example of the syntax adjustment pattern:

print(
    f"Generating sovereign embeddings for {len(texts_to_embed)} "
    f"chunks. Tenant: {tenant_id}..."
)

@fivepanelhat fivepanelhat marked this pull request as ready for review June 28, 2026 23:49
@fivepanelhat fivepanelhat merged commit fdaa551 into main Jun 28, 2026
Copilot AI changed the title [WIP] Fix failing GitHub Actions job build Fix Python 3.10 syntax incompatibilities breaking the build job Jun 28, 2026
Copilot stopped work on behalf of fivepanelhat due to an error June 28, 2026 23:49
Copilot AI requested a review from fivepanelhat June 28, 2026 23:49
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.

2 participants