Skip to content

refactor(misc): drop services_public from graphile plugin docs, compose files, csv-to-pg test#1445

Open
pyramation wants to merge 3 commits into
mainfrom
feat/services-public-hard-break-misc
Open

refactor(misc): drop services_public from graphile plugin docs, compose files, csv-to-pg test#1445
pyramation wants to merge 3 commits into
mainfrom
feat/services-public-hard-break-misc

Conversation

@pyramation

@pyramation pyramation commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Summary

Miscellaneous slice of the hard migration off services_public. The jobs/knative-job-service migration that was originally here has been reverted to main — it must land together with the server-default flip in the core PR #1447, which now owns it.

  • graphile-llm (chat.ts, embedder.ts, plugins/llm-module-plugin.ts, types.ts, README.md): doc references to the llm_module source table now say constructive_routing_public.api_modules. No code path names the schema.
  • graphile-settings/custom-inflector.ts: _schemaPrefix doc example → constructive_routing_public_api -> constructiveRoutingPublicApi; the inflector itself already returns '' for every schema.
  • docker-compose.jobs.yml / docker-compose.local-email.yml: API_EXPOSED_SCHEMASmetaschema_public,constructive_routing_public,constructive_auth_public; API_META_SCHEMAS additionally gets catalog + apps. Routing alone is exposed on purpose: exposing catalog and routing together trips PostGraphile with Attempted to add a second codec named 'apis', since the custom inflector strips schema prefixes and both schemas have an apis table.
  • packages/csv-to-pg: services_public was only an arbitrary example schema name in export.test.ts; renamed to app_public with snapshots re-recorded.

Not changed: __fixtures__/output/** is generated codegen output that still contains schemas/services_public.ts — it belongs to the codegen regeneration follow-up, not a hand edit.

Link to Devin session: https://app.devin.ai/sessions/33ae06cfdc414ea7b15c9795abc0dcb3
Requested by: @pyramation

@pyramation pyramation self-assigned this Jul 25, 2026
@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@blacksmith-sh

This comment has been minimized.

@devin-ai-integration devin-ai-integration Bot changed the title refactor(misc): drop services_public from graphile plugins, jobs fixtures, compose files refactor(misc): drop services_public from graphile plugin docs, compose files, csv-to-pg test Jul 25, 2026
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