My guess is that people who work on small codebases with low-turnover teams (say, Redis or games like The Witness) would say “obviously you have to understand it completely, otherwise you can’t do good work”. I’d also guess that people who work on large codebases with high-turnover teams (say, the Google web search backend or GitHub) would say “obviously you can’t understand it completely, you just have to do the best you can in your local area”.

These are two largely different ways of programming with different methods, practices and cultures. However, the first group is over-represented in online discussion about software engineering. I want to defend the second group against the first.

  • Kissaki@programming.dev
    link
    fedilink
    English
    arrow-up
    1
    ·
    26 minutes ago

    They’re making good, correct arguments, but by not acknowledging and valuing understanding [partials of] your codebase, it ultimately (subjectively) reads like “too far in that direction”. Maybe it makes more sense in a big enterprise setting, but I think we’re talking about software development in general.

    Like many software tools, LLMs are a double-edged sword: they make it harder to construct a detailed mental theory of the software, but they allow you to build a partial theory quickly and they can help you leverage that partial theory more effectively. This is a complex tradeoff that I’m still thinking about.

    Building a theory model is simple, verifying it is not. If possible, assumption verification, automated tests verifying assumptions, manual review of coverate of tests vs assumptions and cross references are important to validate theories.

    In the end, it’s always a weighing of effort, necessity, and more, and weighing factors depend on the environment.

    Other people being allowed to write code in your codebase

    I work in a small team but on a big and long-running project [ecosystem]. I’m fine with people doing stuff without me in the loop as long as I can depend on them for care, due diligence, and understanding. Even then, I often find it valuable for me and the project to be in the loop to be able to raise concerns, ensure quality and longlevity/maintainability.

    I struggle with one colleague in particular, which I can not be sure to be even surface-level correct (they often are, but at times, repeated absolutely obvious wrongdoings), who does not document their understandings intentions or thoughts, and does not seek out understanding from sources or colleagues. Ultimately, we as a team and I personally are responsible for the customer’s product, so this situations is quite frustrating and tense to navigate with some inefficient reviews, concerns, and my other [more productive] tasks.

    I don’t think waving away other colleagues’ contributions is constructive and productive for a project and it’s health, but I do try to move myself away from owning or being involved in everything, even at the cost of quality, certainty, and risk.

    I would assume even for big teams that split into local and wider teams, it’s a concern you should think about and make conscious decisions about. Teams should own their products, they have to navigate and nurture how they work together, and have to continuously weigh effort and consequence.

    they pay you money to adopt their set of engineering values
    […] sometimes you have to write slow code at your job […] Maintaining a theory of the codebase is the same kind of thing.

    They certainly don’t pay me to only adapt to their engineering values. This is probably again about huge companies vs smaller teams. For me, they pay me to bring my engineering values. As a lead, I ensure we have, adjust, and follow engineering values.

  • iocase@lemmy.zip
    link
    fedilink
    arrow-up
    11
    ·
    6 hours ago

    I really don’t want people using this as justification for going from human coding a code base they understand fluently, to AI shitting out loads of files and lines of code they don’t understand…

  • glowie@infosec.pub
    link
    fedilink
    English
    arrow-up
    5
    ·
    8 hours ago

    Completely unrelated, but I see this person’s photo pop up in the link preview frequently around Lemmy and it always reminds me of the type of condescending smile someone would make as they bait you and say “go ahead, punch me”. I’m sure they’re a nice person. Just always trips me up.

  • CombatWombat@feddit.online
    link
    fedilink
    English
    arrow-up
    29
    ·
    13 hours ago

    I find this argument kind of facile. Sure, the codebase that you are working in is 10 million lines and you can’t understand it, but your team owns a subset of that, hopefully with a well-defined interface, and it is reasonable to expect a team to fully understand their subsystem. Whether all submodules are broken out into separate repositories or stored in a big monorepo doesn’t change my expectation that when I sit down in a meeting with the engineers from your team, I expect them to be able to readily answer questions about what the system does and how it does it, even if they had to implement something they would have preferred not to for legal reasons.

    • Kissaki@programming.dev
      link
      fedilink
      English
      arrow-up
      1
      ·
      23 minutes ago

      They made the lib example, and it’s the same there. We have certain expectations on interfaces and we have to assess necessity and risk during onboarding and when upgrading versions.

    • Kissaki@programming.dev
      link
      fedilink
      English
      arrow-up
      2
      ·
      23 minutes ago

      Did they not add their thoughts into the commit descriptions?

      Something helpful like “I have no idea why this works”

  • Skullgrid@lemmy.world
    link
    fedilink
    arrow-up
    8
    ·
    13 hours ago

    There is a salient point being made, but the statement is “I’m here to stand up for the big guy”

  • crystalwalrus@programming.dev
    link
    fedilink
    arrow-up
    7
    ·
    13 hours ago

    Two things: this is phrased as if each change by another developer ruins your theory of the codebase. This is an oversimplification. Changes to the core engine of the codebase can change the theory of the product but many changes are cross cutting concerns. This is why new developers on large codebases are commonly assigned integration work. You develop a theory with a real lower stakes output.

    Second the llm thing. Putting aside hallucinations you could get a start of a theory from an llm but you’ve conditioned yourself to be reliant on the llm. To put things in similar terms of tech job stuff like this article does: prior to llms most teams had at least one SME on the whole codebase. They’d get bombarded with questions and if you had any politeness you wouldn’t bother them with every single question about the code or product as a whole. Occasionally you would need to but there was a natural social forcing function to figure stuff out on your own. Llms are the complete opposite: they tell you “good question” “good insight” etc. brains like short cuts so now there is no forcing function.

  • Valmond@lemmy.dbzer0.com
    link
    fedilink
    arrow-up
    7
    ·
    13 hours ago

    I worked on a 1.8GB c/c++ codebase, split up on 720 something projects, all making a one monolithic software. There were loads of other code too like python, shellscrips, diverse databases, c#, you name it, you could probably dig it up somewhere.

    No one knew half of even a quarter of it all. Some parts were known to many, some parts were feared, some were known to nobody.

    Fantastic software but also having some of the most hilarious bugs provoked when developing.

  • who@feddit.org
    link
    fedilink
    English
    arrow-up
    7
    ·
    edit-2
    11 hours ago

    An argument could be made that the first group is about engineering software, while the second group is about navigating a certain kind of adversity.

    I guess one could call the latter a kind of engineering, but not of software.

    • okwhateverdude@lemmy.world
      link
      fedilink
      English
      arrow-up
      2
      ·
      13 hours ago

      Complexity engineering. You have to develop an intuition of how much chaos your change will introduce. And learn to be fast when shit breaks.