Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Servo TSC Meeting January 2025 #124

Open
mrego opened this issue Jan 14, 2025 · 6 comments
Open

Servo TSC Meeting January 2025 #124

mrego opened this issue Jan 14, 2025 · 6 comments
Labels
tsc-meeting TSC Meeting

Comments

@mrego
Copy link
Member

mrego commented Jan 14, 2025

Agenda

  • Status update
  • New self-hosted runners
  • Whether to replace unpaired surrogates in the DOM by default
  • More coordination on API
  • AI policy review
  • Outreach
  • AOB
@mrego mrego added the tsc-meeting TSC Meeting label Jan 14, 2025
@mrobinson
Copy link
Member

mrobinson commented Jan 15, 2025

I would like to add two agenda items:

@gterzian
Copy link
Member

I would like to add one item: lifting the AI ban for Github copilot.

Here is an overview of my position:

Some background:

  • Current policy bans all use of AI. I've been experimenting with copilot, and found it useful. But, I have to throw away those commits because of the ban.
  • Not only does copilot make me more productive and is it a joy to use, I think using it would also would improve our codebase. Why? Because copilot only works when the code is well documented and based on clear patterns. So copilot gives an incentive to write that kind of code, and it also produces it, which is good for the health of the project.

Below I'll address the various points of the current policy:

  • Maintainer burden: in my experience, works best when you copy/paste a line from a specification as doc comment for almost every line of code produced. The suggestions are then based on existing code doing something similar. This means that the resulting code, a mixture of which is generated and hand edited, ends-up following conventions and is well documented: the code is of higher quality to begin with and therefore easier to review.
  • Correctness and security: I think correctness and security are for a large part about following existing coding conventions, and copilot excels at it. Also, as the name "copilot" implies, you use it to help you with routine stuff, not to make critical decisions.
  • Copyright issues: The code suggestions pick-up on patterns and naming conventions that are specific to the project, so unlikely to come from other code bases.
  • Ethical issues: I think this is something not for the TSC, but for individual contributors to decide for themselves. When it comes to copilot, one could for example read the sustainability report for Microsoft and make a choice.

I propose that we lift the ban specifically for Github copilot.

@Manishearth
Copy link
Member

Manishearth commented Jan 22, 2025

Copyright issues: The code suggestions pick-up on patterns and naming conventions that are specific to the project, so unlikely to come from other code bases.

I think there's an additional aspect: potential downstream users of Servo may have policies around this due to an abundance of caution, and may not be able to use Servo maintained libraries if Servo uses Copilot.

@matlu
Copy link

matlu commented Jan 22, 2025

Interesting, will this open up a new opportunity for how to use Servo?
Would like to hear the thoughts from other TSC attendees.
Note please watch the video before the call. Video is also on YouTube:
https://youtu.be/C25g53PC5QQ?si=cbg1caIHzIecMNg7

https://www.diabrowser.com/

Come build Dia.
AI won’t exist as an app. Or a button. We believe
it’ll be an entirely new environment — built on top of a web browser.
Image

@gterzian
Copy link
Member

Here is a good resource regarding copyright: https://resources.github.com/learn/pathways/copilot/essentials/establishing-trust-in-using-github-copilot/

Both GitHub and Microsoft provide indemnity for their Copilot products.
This framework is based on our confidence in GitHub Copilot’s compliance with IP laws

@gterzian
Copy link
Member

gterzian commented Feb 6, 2025

@Manishearth Thanks for your feedback above.

I wanted to highlight that the issue has been further discussed, and is now up for a vote, at https://servo.zulipchat.com/#narrow/channel/263399-Technical-Steering-Committee/topic/Another.20look.20at.20AI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tsc-meeting TSC Meeting
Projects
None yet
Development

No branches or pull requests

5 participants