Make `.control` file parsing more robust

Open
#81 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
4/5
Estimated time
3-5 days
Newbie friendliness
35/100
Issue type
Refactor
Clarity
Mostly clear
Activity status
Stale
Tech stack
postgresql, rust
Domain
databases

Research direction

Start in models.rs, where the current .control file parsing is implemented, and compare its behavior with PostgreSQL's linked GUC parser. The work is complete when .control parsing follows the referenced parser's logic and handles the cases that the current primitive implementation misses.

Written by the indexing model from the issue text.

Description

Chore

Describe the chore

The current code to parse .control files in the models.rs is primitive. Use a parser that matches the logic in the GUC parser in postgres.

Additional context

N/A

Dominant language
TypeScript
Stars
443
Forks
31
Avg merge
1h 12m
Merged PRs (30d)
10

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from supabase/dbdev

All issues in supabase/dbdev

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.